diff options
| author | Meelis Roos <[email protected]> | 2007-07-31 00:39:41 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2007-07-31 15:39:44 -0700 |
| commit | db7526f925b79293dc3b361a70db7f340870a298 (patch) | |
| tree | a97f7794e713b654c2a9941429a1e1173e3e0f13 /scripts/patch-kernel | |
| parent | ad4a5bb885f1985732426f108d2fe36517dd1939 (diff) | |
fix integer overflow warning in i2o_block
drivers/message/i2o/i2o_block.c: In function 'i2o_block_transfer':
drivers/message/i2o/i2o_block.c:837: warning: integer overflow in expression
msg->u.head[1] = cpu_to_le32(I2O_CMD_PRIVATE << 24 | HOST_TID << 12 | tid);
and I2O_CMD_PRIVATE is defined as 0xFF. This gets "0xFF0100 | tid" and fits
into 32-bit unsigned but not into 32-bit signed integer properly. Target
value is defined as u32 so the claculation does not fit during computation.
Change local variable tid to u32 so the whole expression is of u32 type and
fits well into u32 result.
Signed-off-by: Meelis Roos <[email protected]>
Cc: "Salyzyn, Mark" <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'scripts/patch-kernel')
0 files changed, 0 insertions, 0 deletions