On 02/09/2020 10.05, Philippe Mathieu-Daudé wrote: > Fix an error handling issue reported by Coverity: > > /qemu/tests/qtest/ahci-test.c: 1452 in prepare_iso() > 1444 int fd = mkstemp(cdrom_path); > >>> CID 1432375: Error handling issues (NEGATIVE_RETURNS) > >>> "fd" is passed to a parameter that cannot be negative. > 1452 ret = write(fd, patt, size); > > Reported-by: Coverity (CID 1432375) > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > tests/qtest/ahci-test.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/qtest/ahci-test.c b/tests/qtest/ahci-test.c > index ca4294f44f3..5e1954852e7 100644 > --- a/tests/qtest/ahci-test.c > +++ b/tests/qtest/ahci-test.c > @@ -1443,6 +1443,7 @@ static int prepare_iso(size_t size, unsigned char > **buf, char **name) > ssize_t ret; > int fd = mkstemp(cdrom_path); > > + g_assert(fd != -1);
Should be good enough for a qtest. Reviewed-by: Thomas Huth <th...@redhat.com>