]> git.puffer.fish Git - mirror/frr.git/commitdiff
lib, stream: fix stream sanity checks
authorWenjian Ma <ytmwjian@163.com>
Fri, 19 Jun 2015 02:53:26 +0000 (10:53 +0800)
committerDaniel Walton <dwalton@cumulusnetworks.com>
Thu, 26 May 2016 15:33:30 +0000 (15:33 +0000)
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>
(cherry picked from commit 1ed8ce47b922b71f3b3cdd661e647bbe7ed2eca7)

lib/stream.c

index b0838e7934567c59e2bdb22c8cf1626c6ff2d578..27f13d7698f0baacb684322c6d1599e7b5605491 100644 (file)
@@ -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) ); \