diff options
author | Wenjian Ma <ytmwjian@163.com> | 2015-06-19 10:53:26 +0800 |
---|---|---|
committer | Paul Jakma <paul@quagga.net> | 2015-06-21 12:32:29 +0100 |
commit | 1ed8ce47b922b71f3b3cdd661e647bbe7ed2eca7 (patch) | |
tree | 8991923fd48342254db78275f2776480b824e18d | |
parent | 84c3840c7157438b5cb2e324d184ca08ac07fb41 (diff) | |
download | quagga-1ed8ce47b922b71f3b3cdd661e647bbe7ed2eca7.tar.bz2 quagga-1ed8ce47b922b71f3b3cdd661e647bbe7ed2eca7.tar.xz |
lib, stream: fix stream sanity checks
Because operator "!" has higher priority than "&&",
So we put the "&&" expression in "()" to check both getp and endp.
Acked-by: Donald Sharp <sharpd@cumulusnetworks.com>
-rw-r--r-- | lib/stream.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/stream.c b/lib/stream.c index e13da08b..9c26fea8 100644 --- a/lib/stream.c +++ b/lib/stream.c @@ -60,7 +60,7 @@ #define STREAM_VERIFY_SANE(S) \ do { \ - if ( !(GETP_VALID(S, (S)->getp)) && ENDP_VALID(S, (S)->endp) ) \ + if ( !(GETP_VALID(S, (S)->getp) && ENDP_VALID(S, (S)->endp)) ) \ STREAM_WARN_OFFSETS(S); \ assert ( GETP_VALID(S, (S)->getp) ); \ assert ( ENDP_VALID(S, (S)->endp) ); \ |