[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL v2 09/15] tests/qtest/ahci: Improve error handling (NEGATIVE_RETUR
From: |
Thomas Huth |
Subject: |
[PULL v2 09/15] tests/qtest/ahci: Improve error handling (NEGATIVE_RETURNS) |
Date: |
Wed, 2 Sep 2020 17:49:26 +0200 |
From: Philippe Mathieu-Daudé <philmd@redhat.com>
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é <philmd@redhat.com>
Message-Id: <20200902080552.159806-1-philmd@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@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 ca4294f44f..5e1954852e 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);
g_assert(buf);
g_assert(name);
patt = g_malloc(size);
--
2.18.2
- [PULL v2 00/15] Cirrus-CI improvements, and other CI-related fixes, m68k, Thomas Huth, 2020/09/02
- [PULL v2 02/15] cirrus.yml: Compile FreeBSD with -Werror, Thomas Huth, 2020/09/02
- [PULL v2 03/15] cirrus.yml: Compile macOS with -Werror, Thomas Huth, 2020/09/02
- [PULL v2 01/15] configure: Fix atomic64 test for --enable-werror on macOS, Thomas Huth, 2020/09/02
- [PULL v2 05/15] cirrus.yml: Split FreeBSD job into two parts, Thomas Huth, 2020/09/02
- [PULL v2 08/15] hw/m68k: QOMify the mcf5206 system integration module, Thomas Huth, 2020/09/02
- [PULL v2 09/15] tests/qtest/ahci: Improve error handling (NEGATIVE_RETURNS),
Thomas Huth <=
- [PULL v2 04/15] cirrus.yml: Update the macOS jobs to Catalina, Thomas Huth, 2020/09/02
- [PULL v2 10/15] tests/qtest/tpm: Declare input buffers const and static, Thomas Huth, 2020/09/02
- [PULL v2 07/15] configure: Add system = 'linux' for meson when cross-compiling, Thomas Huth, 2020/09/02
- [PULL v2 06/15] meson: fix keymaps without qemu-keymap, Thomas Huth, 2020/09/02
- [PULL v2 11/15] tests/qtest/ipmi-kcs: Fix assert side-effect, Thomas Huth, 2020/09/02
- [PULL v2 12/15] libqtest: Rename qmp_assert_error_class() to qmp_expect_error_and_unref(), Thomas Huth, 2020/09/02
- [PULL v2 13/15] gitlab/travis: Rework the disabled features tests, Thomas Huth, 2020/09/02
- [PULL v2 14/15] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job, Thomas Huth, 2020/09/02
- [PULL v2 15/15] gitlab-ci.yml: Set artifacts expiration time, Thomas Huth, 2020/09/02
- Re: [PULL v2 00/15] Cirrus-CI improvements, and other CI-related fixes, m68k, Peter Maydell, 2020/09/02