summaryrefslogtreecommitdiffstats
path: root/usr.sbin
diff options
context:
space:
mode:
authorngie <ngie@FreeBSD.org>2016-06-08 13:49:59 +0000
committerngie <ngie@FreeBSD.org>2016-06-08 13:49:59 +0000
commit1e9eae959ce6f13ac755c2da7b1631ee5578b6d7 (patch)
tree7d80465b165dd4774b0923e9fb60ec46d2316eb3 /usr.sbin
parentfac98982e7cde0c7d96e891a460d2d6245e8ea3c (diff)
downloadFreeBSD-src-1e9eae959ce6f13ac755c2da7b1631ee5578b6d7.zip
FreeBSD-src-1e9eae959ce6f13ac755c2da7b1631ee5578b6d7.tar.gz
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
Diffstat (limited to 'usr.sbin')
-rw-r--r--usr.sbin/rpc.yppasswdd/yppasswdd_server.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/usr.sbin/rpc.yppasswdd/yppasswdd_server.c b/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
index 0260e4c..e4754dc 100644
--- a/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
+++ b/usr.sbin/rpc.yppasswdd/yppasswdd_server.c
@@ -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);
OpenPOWER on IntegriCloud