diff options
author | ngie <ngie@FreeBSD.org> | 2016-06-08 13:49:59 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2016-06-08 13:49:59 +0000 |
commit | 1e9eae959ce6f13ac755c2da7b1631ee5578b6d7 (patch) | |
tree | 7d80465b165dd4774b0923e9fb60ec46d2316eb3 /usr.sbin | |
parent | fac98982e7cde0c7d96e891a460d2d6245e8ea3c (diff) | |
download | FreeBSD-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.c | 5 |
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); |