diff options
author | Akinobu Mita <akinobu.mita@gmail.com> | 2013-09-18 12:27:27 (GMT) |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2013-10-25 08:58:12 (GMT) |
commit | 68aee7ba66d390abf48c13791a84f6bce29d6f19 (patch) | |
tree | c382158f4a0fd15743ce6bda063a5bc81e7e89e1 /firmware/whiteheat_loader_debug.HEX | |
parent | 65f72f2a2fe89f072d6a88e5cd69a64270b9c436 (diff) | |
download | linux-68aee7ba66d390abf48c13791a84f6bce29d6f19.tar.xz |
[SCSI] scsi_debug: fix invalid value check for guard module parameter
In the module initialization, invalid value for guard module parameter
is detected by the following check:
if (scsi_debug_guard > 1) {
printk(KERN_ERR "scsi_debug_init: guard must be 0 or 1\n");
return -EINVAL;
}
But this check isn't enough, because the type of scsi_debug_guard is
'int' and scsi_debug_guard could be a negative value.
This fixes it by changing the type of scsi_debug_guard to 'unsigned int'
instead of adding extra check for a negative value.
Reported-by: Joe Perches <joe@perches.com>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Acked-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'firmware/whiteheat_loader_debug.HEX')
0 files changed, 0 insertions, 0 deletions