diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2017-03-23 16:02:18 +0100 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2017-04-04 19:39:39 -0400 |
| commit | 44a5b977128c0ffff0654392b40f4c2ce72a619b (patch) | |
| tree | c374b36ae5ff1b943992b56220b927e05e301d5d /drivers/scsi/qla2xxx/qla_isr.c | |
| parent | 831488669a334ea10f60361bb90f473ce7f655ff (diff) | |
| download | linux-0-day-44a5b977128c0ffff0654392b40f4c2ce72a619b.tar.gz linux-0-day-44a5b977128c0ffff0654392b40f4c2ce72a619b.tar.xz | |
scsi: advansys: fix uninitialized data access
gcc-7.0.1 now warns about a previously unnoticed access of uninitialized
struct members:
drivers/scsi/advansys.c: In function 'AscMsgOutSDTR':
drivers/scsi/advansys.c:3860:26: error: '*((void *)&sdtr_buf+5)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
((ushort)s_buffer[i + 1] << 8) | s_buffer[i]);
^
drivers/scsi/advansys.c:3860:26: error: '*((void *)&sdtr_buf+7)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
drivers/scsi/advansys.c:3860:26: error: '*((void *)&sdtr_buf+5)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
drivers/scsi/advansys.c:3860:26: error: '*((void *)&sdtr_buf+7)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
The code has existed in this exact form at least since v2.6.12, and the
warning seems correct. This uses named initializers to ensure we
initialize all members of the structure.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_isr.c')
0 files changed, 0 insertions, 0 deletions