diff options
| author | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-01-19 15:42:05 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-01-19 15:42:05 -0800 |
| commit | b85df5b86390d17cb31a30745281600ff19fdc60 (patch) | |
| tree | ae33c55ae672062368d3a3193fc94904377f807c /drivers | |
| parent | 171b1609f1ecdbbab1cd2390cbe9d24dc817f54d (diff) | |
scsi_ioctl: only warn about unknown opcodes if they are truly unknown.
We could screw up and warn for opcodes that weren't write-safe and the
user tried to use without having write permissions.
Not so bad in itself, but that also destroyed the cmd type information.
Noticed by Michal Schmidt
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/block/scsi_ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/scsi_ioctl.c b/drivers/block/scsi_ioctl.c index cdb11fc7ad25..689527a89de7 100644 --- a/drivers/block/scsi_ioctl.c +++ b/drivers/block/scsi_ioctl.c @@ -199,7 +199,7 @@ static int verify_command(struct file *file, unsigned char *cmd) return 0; } - if (!(type & CMD_WARNED)) { + if (!type) { cmd_type[cmd[0]] = CMD_WARNED; printk(KERN_WARNING "scsi: unknown opcode 0x%02x\n", cmd[0]); } |
