January 2025 - CI - libvirt List Archives

New Defects reported by Coverity Scan for libvirt
by scan-admin＠coverity.com 12 Jan '25

12 Jan '25

Hi, Please find the latest report on new defect(s) introduced to libvirt found with Coverity Scan. 8 new defect(s) introduced to libvirt found with Coverity Scan. 21 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 8 of 8 defect(s) ** CID 471899: Control flow issues (DEADCODE) /src/util/virstring.c: 137 in virStrToLong_ul() ________________________________________________________________________________________________________ *** CID 471899: Control flow issues (DEADCODE) /src/util/virstring.c: 137 in virStrToLong_ul() 131 /* This one's tricky. We _want_ to allow "-1" as shorthand for 132 * ULONG_MAX regardless of whether long is 32-bit or 64-bit. But 133 * g_ascii_strtoull treats "-1" as ULLONG_MAX, and going from ullong back 134 * to ulong differs depending on the size of ulong. */ 135 if (memchr(s, '-', p - s)) { 136 if (-val > ULONG_MAX) >>> CID 471899: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "err = true;". 137 err = true; 138 else 139 val &= ULONG_MAX; 140 } 141 142 err |= (errno || (!end_ptr && *p) || p == s || (unsigned long) val != val); ** CID 471898: Null pointer dereferences (FORWARD_NULL) /src/remote/remote_driver.c: 900 in doRemoteOpen() ________________________________________________________________________________________________________ *** CID 471898: Null pointer dereferences (FORWARD_NULL) /src/remote/remote_driver.c: 900 in doRemoteOpen() 894 if ((mode = remoteDriverModeTypeFromString(mode_str)) < 0) { 895 virReportError(VIR_ERR_INVALID_ARG, 896 _("Unknown remote mode '%1$s'"), mode_str); 897 goto error; 898 } 899 } else { >>> CID 471898: Null pointer dereferences (FORWARD_NULL) >>> Dereferencing null pointer "conn->uri". 900 if (inside_daemon && !conn->uri->server) { 901 mode = REMOTE_DRIVER_MODE_DIRECT; 902 } else { 903 mode = REMOTE_DRIVER_MODE_AUTO; 904 } 905 } ** CID 471897: Resource leaks (RESOURCE_LEAK) /src/ch/ch_monitor.c: 737 in virCHMonitorPutNoContent() ________________________________________________________________________________________________________ *** CID 471897: Resource leaks (RESOURCE_LEAK) /src/ch/ch_monitor.c: 737 in virCHMonitorPutNoContent() 731 domainLogContextWrite(logCtxt, "Response = %s\n", data.content); 732 } 733 734 if (responseCode == 200 || responseCode == 204) 735 ret = 0; 736 >>> CID 471897: Resource leaks (RESOURCE_LEAK) >>> Variable "data" going out of scope leaks the storage "data.content" points to. 737 return ret; 738 } 739 740 static int 741 virCHMonitorGet(virCHMonitor *mon, const char *endpoint, virJSONValue **response) 742 { ** CID 471896: Integer handling issues (INTEGER_OVERFLOW) /src/conf/capabilities.c: 1036 in virCapabilitiesFormatMemoryBandwidth() ________________________________________________________________________________________________________ *** CID 471896: Integer handling issues (INTEGER_OVERFLOW) /src/conf/capabilities.c: 1036 in virCapabilitiesFormatMemoryBandwidth() 1030 1031 virBufferAddLit(buf, "<memory_bandwidth>\n"); 1032 virBufferAdjustIndent(buf, 2); 1033 1034 for (i = 0; i < memBW->nnodes; i++) { 1035 g_auto(virBuffer) attrBuf = VIR_BUFFER_INITIALIZER; >>> CID 471896: Integer handling issues (INTEGER_OVERFLOW) >>> Expression "buf->indent + 2", where "buf->indent" is known to be equal to 2147483647, overflows the type of "buf->indent + 2", which is type "int". 1036 g_auto(virBuffer) childrenBuf = VIR_BUFFER_INIT_CHILD(buf); 1037 virCapsHostMemBWNode *node = memBW->nodes[i]; 1038 virResctrlInfoMemBWPerNode *control = &node->control; 1039 g_autofree char *cpus_str = virBitmapFormat(node->cpus); 1040 1041 if (!cpus_str) ** CID 471895: Resource leaks (RESOURCE_LEAK) ________________________________________________________________________________________________________ *** CID 471895: Resource leaks (RESOURCE_LEAK) /src/rpc/virnetclientprogram.c: 388 in virNetClientProgramCall() 382 383 virNetMessageFree(msg); 384 385 return 0; 386 387 error: >>> CID 471895: Resource leaks (RESOURCE_LEAK) >>> Freeing "msg" without freeing its pointer field "fds" leaks the storage that "fds" points to. 388 virNetMessageFree(msg); 389 if (infds && ninfds) { 390 for (i = 0; i < *ninfds; i++) 391 VIR_FORCE_CLOSE((*infds)[i]); 392 } 393 return -1; ** CID 471894: Null pointer dereferences (NULL_RETURNS) ________________________________________________________________________________________________________ *** CID 471894: Null pointer dereferences (NULL_RETURNS) /tests/testutilsqemuschema.c: 360 in testQEMUSchemaValidateObject() 354 if (virJSONValueObjectForeachKeyValue(obj, 355 testQEMUSchemaValidateObjectMember, 356 &data) < 0) 357 return -1; 358 359 /* check missing mandatory values */ >>> CID 471894: Null pointer dereferences (NULL_RETURNS) >>> Dereferencing a pointer that might be "NULL" "data.rootmembers" when calling "virJSONValueArrayForeachSteal". 360 if (virJSONValueArrayForeachSteal(data.rootmembers, 361 testQEMUSchemaValidateObjectMandatoryMember, 362 &data) < 0) { 363 return -2; 364 } 365 ** CID 471893: Integer handling issues (INTEGER_OVERFLOW) /src/conf/capabilities.c: 951 in virCapabilitiesFormatCaches() ________________________________________________________________________________________________________ *** CID 471893: Integer handling issues (INTEGER_OVERFLOW) /src/conf/capabilities.c: 951 in virCapabilitiesFormatCaches() 945 946 virBufferAddLit(buf, "<cache>\n"); 947 virBufferAdjustIndent(buf, 2); 948 949 for (i = 0; i < cache->nbanks; i++) { 950 g_auto(virBuffer) attrBuf = VIR_BUFFER_INITIALIZER; >>> CID 471893: Integer handling issues (INTEGER_OVERFLOW) >>> Expression "buf->indent + 2", where "buf->indent" is known to be equal to 2147483647, overflows the type of "buf->indent + 2", which is type "int". 951 g_auto(virBuffer) childrenBuf = VIR_BUFFER_INIT_CHILD(buf); 952 virCapsHostCacheBank *bank = cache->banks[i]; 953 g_autofree char *cpus_str = virBitmapFormat(bank->cpus); 954 const char *unit = NULL; 955 unsigned long long short_size = virFormatIntPretty(bank->size, &unit); 956 ** CID 471892: Integer handling issues (INTEGER_OVERFLOW) /src/conf/snapshot_conf.c: 860 in virDomainSnapshotDefFormatInternal() ________________________________________________________________________________________________________ *** CID 471892: Integer handling issues (INTEGER_OVERFLOW) /src/conf/snapshot_conf.c: 860 in virDomainSnapshotDefFormatInternal() 854 } 855 virBufferAdjustIndent(buf, -2); 856 virBufferAddLit(buf, "</disks>\n"); 857 } 858 859 if (def->nrevertdisks > 0) { >>> CID 471892: Integer handling issues (INTEGER_OVERFLOW) >>> Expression "buf->indent + 2", where "buf->indent" is known to be equal to 2147483647, overflows the type of "buf->indent + 2", which is type "int". 860 g_auto(virBuffer) childBuf = VIR_BUFFER_INIT_CHILD(buf); 861 862 for (i = 0; i < def->nrevertdisks; i++) { 863 if (virDomainSnapshotDiskDefFormat(&childBuf, &def->revertdisks[i], xmlopt) < 0) 864 return -1; 865 } ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/libvirt?tab=overview

1 0

libvirt | Failed pipeline for master | f35bddae
by GitLab 12 Jan '25

12 Jan '25

Pipeline #1621269851 has failed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: f35bddae ( https://gitlab.com/libvirt/libvirt/-/commit/f35bddae90b6daf4cf76ac2f95dfd7d… ) Commit Message: docs: standard linux bridges now support vlans ... Commit Author: Leigh ( https://gitlab.com/leighbb ) Committed by: Laine Stump ( https://gitlab.com/lainestump ) Pipeline #1621269851 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1621269851 ) triggered by Daniel P_ Berrangé ( https://gitlab.com/berrange ) had 1 failed job. Job #8828682416 ( https://gitlab.com/libvirt/libvirt/-/jobs/8828682416/raw ) Stage: integration_tests Name: centos-stream-9-tests -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | f35bddae
by GitLab 12 Jan '25

12 Jan '25

Pipeline #1621218392 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: f35bddae ( https://gitlab.com/libvirt/libvirt/-/commit/f35bddae90b6daf4cf76ac2f95dfd7d… ) Commit Message: docs: standard linux bridges now support vlans ... Commit Author: Leigh ( https://gitlab.com/leighbb ) Committed by: Laine Stump ( https://gitlab.com/lainestump ) Pipeline #1621218392 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1621218392 ) triggered by Laine Stump ( https://gitlab.com/lainestump ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | d51179fa
by GitLab 11 Jan '25

11 Jan '25

Pipeline #1620495551 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: d51179fa ( https://gitlab.com/libvirt/libvirt/-/commit/d51179fa82448f4720f1645f0b7100d… ) Commit Message: qemu: snapshot: delete disk image only if paren... Commit Author: Pavel Hrdina ( https://gitlab.com/phrdina ) Pipeline #1620495551 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1620495551 ) triggered by Daniel P_ Berrangé ( https://gitlab.com/berrange ) successfully completed 54 jobs in 3 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | d51179fa
by GitLab 10 Jan '25

10 Jan '25

Pipeline #1620040425 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: d51179fa ( https://gitlab.com/libvirt/libvirt/-/commit/d51179fa82448f4720f1645f0b7100d… ) Commit Message: qemu: snapshot: delete disk image only if paren... Commit Author: Pavel Hrdina ( https://gitlab.com/phrdina ) Pipeline #1620040425 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1620040425 ) triggered by Pavel Hrdina ( https://gitlab.com/hrdina ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Failed pipeline for master | 30b0e553
by GitLab 10 Jan '25

10 Jan '25

Pipeline #1618867481 has failed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: 30b0e553 ( https://gitlab.com/libvirt/libvirt/-/commit/30b0e553bcb4164d3ddc67a1038b680… ) Commit Message: docs: Clarify documentation of host-model CPU m... Commit Author: Jiri Denemark ( https://gitlab.com/jirkade ) Pipeline #1618867481 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1618867481 ) triggered by Daniel P_ Berrangé ( https://gitlab.com/berrange ) had 2 failed jobs. Job #8814357692 ( https://gitlab.com/libvirt/libvirt/-/jobs/8814357692/raw ) Stage: integration_tests Name: centos-stream-9-tests Job #8814357696 ( https://gitlab.com/libvirt/libvirt/-/jobs/8814357696/raw ) Stage: integration_tests Name: fedora-39-tests -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | 30b0e553
by GitLab 09 Jan '25

09 Jan '25

Pipeline #1618104486 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: 30b0e553 ( https://gitlab.com/libvirt/libvirt/-/commit/30b0e553bcb4164d3ddc67a1038b680… ) Commit Message: docs: Clarify documentation of host-model CPU m... Commit Author: Jiri Denemark ( https://gitlab.com/jirkade ) Pipeline #1618104486 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1618104486 ) triggered by Jiri Denemark ( https://gitlab.com/jirkade ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | 4c7bb1cc
by GitLab 09 Jan '25

09 Jan '25

Pipeline #1617742221 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: 4c7bb1cc ( https://gitlab.com/libvirt/libvirt/-/commit/4c7bb1cc4a5f5696893428a4ad0fa8b… ) Commit Message: po: Refresh potfile for v11.0.0 Signed-off-by:... Commit Author: Jiri Denemark ( https://gitlab.com/jirkade ) Pipeline #1617742221 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1617742221 ) triggered by Jiri Denemark ( https://gitlab.com/jirkade ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | caa10431
by GitLab 09 Jan '25

09 Jan '25

Pipeline #1617599309 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: caa10431 ( https://gitlab.com/libvirt/libvirt/-/commit/caa10431cdd1aa476637ff721f1947c… ) Commit Message: util: don't attempt to acquire logind inhibitor... Commit Author: Daniel P_ Berrangé ( https://gitlab.com/berrange ) Pipeline #1617599309 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1617599309 ) triggered by Daniel P_ Berrangé ( https://gitlab.com/berrange ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0

libvirt | Successful pipeline for master | a28a5b67
by GitLab 09 Jan '25

09 Jan '25

Pipeline #1617594308 has passed! Project: libvirt ( https://gitlab.com/libvirt/libvirt ) Branch: master ( https://gitlab.com/libvirt/libvirt/-/commits/master ) Commit: a28a5b67 ( https://gitlab.com/libvirt/libvirt/-/commit/a28a5b675d5362dd9380d93630d282b… ) Commit Message: rpc: log details of OS signals received by daem... Commit Author: Daniel P_ Berrangé ( https://gitlab.com/berrange ) Pipeline #1617594308 ( https://gitlab.com/libvirt/libvirt/-/pipelines/1617594308 ) triggered by Daniel P_ Berrangé ( https://gitlab.com/berrange ) successfully completed 54 jobs in 4 stages. -- You're receiving this email because of your account on gitlab.com.

1 0