diff options
| author | Philippe Mathieu-Daudé <philmd@redhat.com> | 2020-06-29 09:08:58 +0200 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2020-07-07 12:38:50 +0200 |
| commit | 8f4d9555823636c329cb40afc6f36969ec5d35e4 (patch) | |
| tree | 7611f209465a4799e467d7a293ce5e972f7ed3a9 /hw/display/virtio-gpu-pci.c | |
| parent | 428a37770406346dac6d355a70ea5a8916aecb1f (diff) | |
| download | focaccia-qemu-8f4d9555823636c329cb40afc6f36969ec5d35e4.tar.gz focaccia-qemu-8f4d9555823636c329cb40afc6f36969ec5d35e4.zip | |
util/qemu-option: Document the get_opt_value() function
Coverity noticed commit 950c4e6c94 introduced a dereference before null check in get_opt_value (CID1391003): In get_opt_value: All paths that lead to this null pointer comparison already dereference the pointer earlier (CWE-476) We fixed this in commit 6e3ad3f0e31, but relaxed the check in commit 0c2f6e7ee99 because "No callers of get_opt_value() pass in a NULL for the 'value' parameter". Since this function is publicly exposed, it risks new users to do the same error again. Avoid that documenting the 'value' argument must not be NULL. Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Message-Id: <20200629070858.19850-1-philmd@redhat.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'hw/display/virtio-gpu-pci.c')
0 files changed, 0 insertions, 0 deletions