NuKVM/libvirt
3
0
Fork 0
mirror of https://github.com/libvirt/libvirt.git synced 2026-09-03 20:53:04 -05:00
Code Issues Packages Projects Releases Wiki Activity

More coverity findings addressed

Browse Source 
More bug extermination in the category of:

Error: CHECKED_RETURN:

/libvirt/src/conf/network_conf.c:595:
check_return: Calling function "virAsprintf" without checking return value (as is done elsewhere 515 out of 543 times).

/libvirt/src/qemu/qemu_process.c:2780:
unchecked_value: No check of the return value of "virAsprintf(&msg, "was paused (%s)", virDomainPausedReasonTypeToString(reason))".

/libvirt/tests/commandtest.c:809:
check_return: Calling function "setsid" without checking return value (as is done elsewhere 4 out of 5 times).

/libvirt/tests/commandtest.c:830:
unchecked_value: No check of the return value of "virTestGetDebug()".

/libvirt/tests/commandtest.c:831:
check_return: Calling function "virTestGetVerbose" without checking return value (as is done elsewhere 41 out of 42 times).

/libvirt/tests/commandtest.c:833:
check_return: Calling function "virInitialize" without checking return value (as is done elsewhere 18 out of 21 times).


One note about the error in commandtest line 809: setsid() seems to fail when running the test -- could be removed ?
This commit is contained in:
Stefan Berger
2012-04-27 17:25:35 -04:00
committed by Stefan Berger
parent 2eabac008e
commit 59b935f5ae
3 changed files with 9 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/commandtest.c
+5 -4
View File
@@ -806,7 +806,7 @@ mymain(void)
return EXIT_FAILURE;
setpgid(0, 0);
setsid();
ignore_value(setsid());
/* Our test expects particular fd values; to get that, we must not
* leak fds that we inherited from a lazy parent. At the same
@@ -827,10 +827,11 @@ mymain(void)
/* Prime the debug/verbose settings from the env vars,
* since we're about to reset 'environ' */
virTestGetDebug();
virTestGetVerbose();
ignore_value(virTestGetDebug());
ignore_value(virTestGetVerbose());
virInitialize();
if (virInitialize() < 0)
return EXIT_FAILURE;
/* Phase two of killing interfering fds; see above. */
fd = 3;