Since the expression "sizeof(*sense) + sizeof(*sense) % 4" did not make
authorbvassche <bvassche@d57e44dd-8a1f-0410-8b47-8ef2f437770f>
Sun, 15 Nov 2009 13:14:40 +0000 (13:14 +0000)
committerbvassche <bvassche@d57e44dd-8a1f-0410-8b47-8ef2f437770f>
Sun, 15 Nov 2009 13:14:40 +0000 (13:14 +0000)
sense, replaced this expression by "sizeof(*sense)" and a BUILD_BUG_ON()
statement that verifies that sizeof(*sense) is a multiple of four. Should
the first expression have been "roundup(sizeof(*sense), 4)" ?

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@1336 d57e44dd-8a1f-0410-8b47-8ef2f437770f

srpt/src/ib_srpt.c

index 37aeab0..0d6e82f 100644 (file)
@@ -1146,8 +1146,9 @@ static int srpt_build_cmd_rsp(struct srpt_rdma_ch *ch,
        int sense_data_len;
        int resp_len;
 
-       sense_data_len = (s_key == NO_SENSE) ? 0
-               : sizeof(*sense) + sizeof(*sense) % 4;
+       BUILD_BUG_ON((sizeof(*sense) % 4) != 0);
+
+       sense_data_len = (s_key == NO_SENSE) ? 0 : sizeof(*sense);
        resp_len = sizeof(*srp_rsp) + sense_data_len;
 
        srp_rsp = ioctx->buf;