diff options
author | Harvey Harrison <harvey.harrison@gmail.com> | 2008-07-18 00:33:04 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2008-07-18 15:18:35 -0400 |
commit | 5108b27651727b5aba0826e8fd7be71b42428701 (patch) | |
tree | 993f3f060d659f3c96644aab049fb3c02d9dc37c | |
parent | ad1060c89cfe451de849373d98e42fad58dd25ae (diff) |
nfsd: nfs4xdr.c do-while is not a compound statement
The WRITEMEM macro produces sparse warnings of the form:
fs/nfsd/nfs4xdr.c:2668:2: warning: do-while statement is not a compound statement
Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
Cc: Benny Halevy <bhalevy@panasas.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
-rw-r--r-- | fs/nfsd/nfs4xdr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c index d4b9d09a6683..14ba4d9b2859 100644 --- a/fs/nfsd/nfs4xdr.c +++ b/fs/nfsd/nfs4xdr.c | |||
@@ -1160,11 +1160,11 @@ nfsd4_decode_compound(struct nfsd4_compoundargs *argp) | |||
1160 | *p++ = htonl((u32)((n) >> 32)); \ | 1160 | *p++ = htonl((u32)((n) >> 32)); \ |
1161 | *p++ = htonl((u32)(n)); \ | 1161 | *p++ = htonl((u32)(n)); \ |
1162 | } while (0) | 1162 | } while (0) |
1163 | #define WRITEMEM(ptr,nbytes) do if (nbytes > 0) { \ | 1163 | #define WRITEMEM(ptr,nbytes) do { if (nbytes > 0) { \ |
1164 | *(p + XDR_QUADLEN(nbytes) -1) = 0; \ | 1164 | *(p + XDR_QUADLEN(nbytes) -1) = 0; \ |
1165 | memcpy(p, ptr, nbytes); \ | 1165 | memcpy(p, ptr, nbytes); \ |
1166 | p += XDR_QUADLEN(nbytes); \ | 1166 | p += XDR_QUADLEN(nbytes); \ |
1167 | } while (0) | 1167 | }} while (0) |
1168 | #define WRITECINFO(c) do { \ | 1168 | #define WRITECINFO(c) do { \ |
1169 | *p++ = htonl(c.atomic); \ | 1169 | *p++ = htonl(c.atomic); \ |
1170 | *p++ = htonl(c.before_ctime_sec); \ | 1170 | *p++ = htonl(c.before_ctime_sec); \ |