SH> That's sucky...  yeah i would say a comment, though of course it
SH> could be one of those cases where everyone but me already knows...

Here's a nice fix brought to us by Mr. Lynch...

-- 
Dan Smith
IBM Linux Technology Center
email: [email protected]

Signed-off-by: Dan Smith <[email protected]>

diff --git a/include/linux/checkpoint.h b/include/linux/checkpoint.h
index 59ec563..a8d758f 100644
--- a/include/linux/checkpoint.h
+++ b/include/linux/checkpoint.h
@@ -138,12 +138,14 @@ extern int cr_read_file(struct cr_ctx *ctx, int objref);
  */
 #define CR_COPY_ARRAY(op, SAVE, LIVE, count)                           \
        do {                                                            \
+               (void)__must_be_array(SAVE);                            \
+               (void)__must_be_array(LIVE);                            \
                BUILD_BUG_ON(sizeof(*SAVE) != sizeof(*LIVE));           \
                if (op == CR_CPT)                                       \
                        memcpy(SAVE, LIVE, count * sizeof(*SAVE));      \
                else                                                    \
                        memcpy(LIVE, SAVE, count * sizeof(*SAVE));      \
-       } while (__must_be_array(SAVE) && __must_be_array(LIVE) && 0)
+       } while (0)
 
 
 #define cr_debug(fmt, args...)  \



_______________________________________________
Containers mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
[email protected]
https://openvz.org/mailman/listinfo/devel

Reply via email to