]> git.proxmox.com Git - mirror_qemu.git/commit
scsi: Silence gcc warning
authorEric Blake <eblake@redhat.com>
Tue, 9 Feb 2021 15:23:50 +0000 (09:23 -0600)
committerLaurent Vivier <laurent@vivier.eu>
Tue, 9 Mar 2021 20:21:54 +0000 (21:21 +0100)
commite91bae8e98a6438156752dfbe9c0e2494d4b80f6
tree668216b4d43f0f9ec607e7245773d0927ce446b7
parentcba42d61a379bc1c983ddb39d479de3581d2d578
scsi: Silence gcc warning

On Fedora 33, gcc 10.2.1 notes that scsi_cdb_length(buf) can set
len==-1, which in turn overflows g_malloc():

[5/5] Linking target qemu-system-x86_64
In function ‘scsi_disk_new_request_dump’,
    inlined from ‘scsi_new_request’ at ../hw/scsi/scsi-disk.c:2608:9:
../hw/scsi/scsi-disk.c:2582:19: warning: argument 1 value ‘18446744073709551612’ exceeds maximum object size 9223372036854775807 [-Walloc-size-larger-than=]
 2582 |     line_buffer = g_malloc(len * 5 + 1);
      |                   ^

Silence it with a decent assertion, since we only convert a buffer to
bytes when we have a valid cdb length.

Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20210209152350.207958-1-eblake@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
hw/scsi/scsi-disk.c