nfsd: nfs4xdr.c do-while is not a compound statement
authorHarvey Harrison <harvey.harrison@gmail.com>
Fri, 18 Jul 2008 04:33:04 +0000 (21:33 -0700)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Fri, 18 Jul 2008 19:18:35 +0000 (15:18 -0400)
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>
fs/nfsd/nfs4xdr.c

index d4b9d09a668388146c432d5c471338b2f06bbf0c..14ba4d9b285913f8aea89eae504908eebc56f435 100644 (file)
@@ -1160,11 +1160,11 @@ nfsd4_decode_compound(struct nfsd4_compoundargs *argp)
        *p++ = htonl((u32)((n) >> 32));                         \
        *p++ = htonl((u32)(n));                                 \
 } while (0)
-#define WRITEMEM(ptr,nbytes)     do if (nbytes > 0) {          \
+#define WRITEMEM(ptr,nbytes)     do { if (nbytes > 0) {                \
        *(p + XDR_QUADLEN(nbytes) -1) = 0;                      \
        memcpy(p, ptr, nbytes);                                 \
        p += XDR_QUADLEN(nbytes);                               \
-} while (0)
+}} while (0)
 #define WRITECINFO(c)          do {                            \
        *p++ = htonl(c.atomic);                                 \
        *p++ = htonl(c.before_ctime_sec);                               \