]> Git Repo - qemu.git/commit - scsi/qemu-pr-helper.c
scsi/qemu-pr-helper: Fix out-of-bounds access to trnptid_list[]
authorChristophe de Dinechin <[email protected]>
Fri, 28 Feb 2020 15:00:59 +0000 (16:00 +0100)
committerPaolo Bonzini <[email protected]>
Mon, 16 Mar 2020 21:07:24 +0000 (22:07 +0100)
commit4ce1e15fbc7266a108a7c77a3962644b3935346e
tree998aa0959cb0af451b86aacea5b81a7a64bab538
parenta98135f727595382e200d04c2996e868b7925a01
scsi/qemu-pr-helper: Fix out-of-bounds access to trnptid_list[]

Compile error reported by gcc 10.0.1:

scsi/qemu-pr-helper.c: In function ‘multipath_pr_out’:
scsi/qemu-pr-helper.c:523:32: error: array subscript <unknown> is outside array bounds of ‘struct transportid *[0]’ [-Werror=array-bounds]
  523 |             paramp.trnptid_list[paramp.num_transportid++] = id;
      |             ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from scsi/qemu-pr-helper.c:36:
/usr/include/mpath_persist.h:168:22: note: while referencing ‘trnptid_list’
  168 |  struct transportid *trnptid_list[];
      |                      ^~~~~~~~~~~~
scsi/qemu-pr-helper.c:424:35: note: defined here ‘paramp’
  424 |     struct prout_param_descriptor paramp;
      |                                   ^~~~~~

This highlights an actual implementation issue in function multipath_pr_out.
The variable paramp is declared with type `struct prout_param_descriptor`,
which is a struct terminated by an empty array in mpath_persist.h:

        struct transportid *trnptid_list[];

That empty array was filled with code that looked like that:

        trnptid_list[paramp.descr.num_transportid++] = id;

This is an actual out-of-bounds access.

The fix is to malloc `paramp`.

Signed-off-by: Christophe de Dinechin <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
scsi/qemu-pr-helper.c
This page took 0.027676 seconds and 4 git commands to generate.