ivshmem: add missing error exit(2)
If the user fails to specify 'chardev' or 'shm' then we cannot continue. Exit right away so that we don't invoke shm_open(3) with a NULL pointer. It would be nice to replace exit(1) with error returns in the PCI device .init() function, but leave that for another patch since exit(1) is currently used elsewhere. Spotted by Coverity. Cc: Cam Macdonell <cam@cs.ualberta.ca> Cc: qemu-stable@nongnu.org Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
This commit is contained in:
parent
c5633d998a
commit
baefb8bf8e
@ -735,6 +735,7 @@ static int pci_ivshmem_init(PCIDevice *dev)
|
||||
|
||||
if (s->shmobj == NULL) {
|
||||
fprintf(stderr, "Must specify 'chardev' or 'shm' to ivshmem\n");
|
||||
exit(1);
|
||||
}
|
||||
|
||||
IVSHMEM_DPRINTF("using shm_open (shm object = %s)\n", s->shmobj);
|
||||
|
Loading…
Reference in New Issue
Block a user