Author: ngie
Date: Wed Jun  8 13:50:48 2016
New Revision: 301620
URL: https://svnweb.freebsd.org/changeset/base/301620

Log:
  MFstable/10 r301619:
  
  MFC r300430:
  
  Wrap EXPAND(..) macro with a do-while(0) loop and put a single statement on 
each line
  
  As a positive side-effect, this eliminates the double semicolons reported by 
Coverity:
  the macro contained a trailing semicolon, in addition to the semicolon placed 
on
  each line where EXPAND(..) was called.
  
  CID: 1194269

Modified:
  stable/9/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/usr.sbin/   (props changed)
  stable/9/usr.sbin/rpc.yppasswdd/   (props changed)

Modified: stable/9/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
==============================================================================
--- stable/9/usr.sbin/rpc.yppasswdd/yppasswdd_server.c  Wed Jun  8 13:49:59 
2016        (r301619)
+++ stable/9/usr.sbin/rpc.yppasswdd/yppasswdd_server.c  Wed Jun  8 13:50:48 
2016        (r301620)
@@ -103,7 +103,10 @@ copy_yp_pass(char *p, int x, int m)
        }
 
        t = buf;
-#define EXPAND(e)       e = t; while ((*t++ = *p++));
+#define EXPAND(e) do { \
+       e = t; \
+       while ((*t++ = *p++)); \
+} while (0)
         EXPAND(yp_password.pw_name);
        yp_password.pw_fields |= _PWF_NAME;
         EXPAND(yp_password.pw_passwd);
_______________________________________________
svn-src-stable-9@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-9
To unsubscribe, send any mail to "svn-src-stable-9-unsubscr...@freebsd.org"

Reply via email to