Author: ngie
Date: Wed Jun  8 13:49:59 2016
New Revision: 301619
URL: https://svnweb.freebsd.org/changeset/base/301619

Log:
  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/10/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
==============================================================================
--- stable/10/usr.sbin/rpc.yppasswdd/yppasswdd_server.c Wed Jun  8 13:45:02 
2016        (r301618)
+++ stable/10/usr.sbin/rpc.yppasswdd/yppasswdd_server.c Wed Jun  8 13:49:59 
2016        (r301619)
@@ -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-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to