Bug 851401 - libvirtd memory leak
libvirtd memory leak
Status: CLOSED INSUFFICIENT_DATA
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: libvirt (Show other bugs)
6.4
Unspecified Linux
high Severity high
: rc
: ---
Assigned To: Gunannan Ren
Virtualization Bugs
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-24 01:39 EDT by yanbing du
Modified: 2012-10-08 10:19 EDT (History)
8 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-10-08 10:19:43 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description yanbing du 2012-08-24 01:39:41 EDT
Description of problem:
valgrind defects some errors and memory leak.


Version-Release number of selected component (if applicable):
libvirt-0.10.0-0rc1.el6.x86_64


How reproducible:
100%

Steps to Reproduce:
1. Stop libvirtd service.
2. run # valgrind libvirtd -d


Actual results:
[root@intel-w3520-8-2 ~]# valgrind libvirtd -d
==26158== Memcheck, a memory error detector
==26158== Copyright (C) 2002-2010, and GNU GPL'd, by Julian Seward et al.
==26158== Using Valgrind-3.6.0 and LibVEX; rerun with -h for copyright info
==26158== Command: libvirtd -d
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x334AD2E2E0: __strcasestr_sse42 (in /lib64/libc-2.12.so)
==26158==    by 0x3361404004: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x334AD2E2E4: __strcasestr_sse42 (in /lib64/libc-2.12.so)
==26158==    by 0x3361404004: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x334AD2E2EC: __strcasestr_sse42 (in /lib64/libc-2.12.so)
==26158==    by 0x3361404004: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x336140400D: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x3361404016: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Use of uninitialised value of size 8
==26158==    at 0x3361404049: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x3361404047: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x3361404057: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Conditional jump or move depends on uninitialised value(s)
==26158==    at 0x336140408F: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26158== Use of uninitialised value of size 8
==26158==    at 0x3361404091: numa_node_size64 (in /usr/lib64/libnuma.so.1)
==26158==    by 0x3361405A48: ??? (in /usr/lib64/libnuma.so.1)
==26158==    by 0x334A40E534: _dl_init (in /lib64/ld-2.12.so)
==26158==    by 0x334A400B39: ??? (in /lib64/ld-2.12.so)
==26158==    by 0x1: ???
==26158==    by 0x7FF00075E: ???
==26158==    by 0x7FF000767: ???
==26158== 
==26159== 
==26159== HEAP SUMMARY:
==26159==     in use at exit: 127,514 bytes in 1,376 blocks
==26159==   total heap usage: 2,854 allocs, 1,478 frees, 303,764 bytes allocated
==26159== 
==26159== LEAK SUMMARY:
==26159==    definitely lost: 0 bytes in 0 blocks
==26159==    indirectly lost: 0 bytes in 0 blocks
==26159==      possibly lost: 349 bytes in 18 blocks
==26159==    still reachable: 127,165 bytes in 1,358 blocks
==26159==         suppressed: 0 bytes in 0 blocks
==26159== Rerun with --leak-check=full to see details of leaked memory
==26159== 
==26159== For counts of detected and suppressed errors, rerun with: -v
==26159== Use --track-origins=yes to see where uninitialised values come from
==26159== ERROR SUMMARY: 45 errors from 10 contexts (suppressed: 8 from 6)
==26158== 
==26158== HEAP SUMMARY:
==26158==     in use at exit: 127,514 bytes in 1,376 blocks
==26158==   total heap usage: 2,850 allocs, 1,474 frees, 303,506 bytes allocated
==26158== 
==26158== LEAK SUMMARY:
==26158==    definitely lost: 232 bytes in 1 blocks
==26158==    indirectly lost: 58 bytes in 5 blocks
==26158==      possibly lost: 349 bytes in 18 blocks
==26158==    still reachable: 126,875 bytes in 1,352 blocks
==26158==         suppressed: 0 bytes in 0 blocks
==26158== Rerun with --leak-check=full to see details of leaked memory
==26158== 
==26158== For counts of detected and suppressed errors, rerun with: -v
==26158== Use --track-origins=yes to see where uninitialised values come from
==26158== ERROR SUMMARY: 45 errors from 10 contexts (suppressed: 8 from 6)
[root@intel-w3520-8-2 ~]# ==26160== Thread 12:
==26160== Invalid read of size 8
==26160==    at 0x334AD3228D: __strspn_sse42 (in /lib64/libc-2.12.so)
==26160==    by 0x30EB661352: virDiskNameToIndex (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB6819BE: virDomainDiskInsertPreAlloced (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69B23D: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69F8A8: virDomainLoadAllConfigs (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x449119: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB6E451A: virStateInitialize (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x421300: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB65EBF8: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x334B007850: start_thread (in /lib64/libpthread-2.12.so)
==26160==    by 0x334ACE767C: clone (in /lib64/libc-2.12.so)
==26160==  Address 0x4e4b970 is 0 bytes inside a block of size 4 alloc'd
==26160==    at 0x4A05FDE: malloc (vg_replace_malloc.c:236)
==26160==    by 0x3351CA695D: xmlStrndup (in /usr/lib64/libxml2.so.2.7.6)
==26160==    by 0x30EB68D41E: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69B220: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69F8A8: virDomainLoadAllConfigs (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x449119: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB6E451A: virStateInitialize (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x421300: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB65EBF8: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x334B007850: start_thread (in /lib64/libpthread-2.12.so)
==26160==    by 0x334ACE767C: clone (in /lib64/libc-2.12.so)
==26160== 
==26160== Invalid read of size 8
==26160==    at 0x334AD3228D: __strspn_sse42 (in /lib64/libc-2.12.so)
==26160==    by 0x30EB661352: virDiskNameToIndex (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB6819CA: virDomainDiskInsertPreAlloced (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69B23D: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69F8A8: virDomainLoadAllConfigs (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x449119: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB6E451A: virStateInitialize (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x421300: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB65EBF8: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x334B007850: start_thread (in /lib64/libpthread-2.12.so)
==26160==    by 0x334ACE767C: clone (in /lib64/libc-2.12.so)
==26160==  Address 0x4e4cb60 is 0 bytes inside a block of size 4 alloc'd
==26160==    at 0x4A05FDE: malloc (vg_replace_malloc.c:236)
==26160==    by 0x3351CA695D: xmlStrndup (in /usr/lib64/libxml2.so.2.7.6)
==26160==    by 0x30EB68D41E: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69B220: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x30EB69F8A8: virDomainLoadAllConfigs (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x449119: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB6E451A: virStateInitialize (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x421300: ??? (in /usr/sbin/libvirtd)
==26160==    by 0x30EB65EBF8: ??? (in /usr/lib64/libvirt.so.0.10.0)
==26160==    by 0x334B007850: start_thread (in /lib64/libpthread-2.12.so)
==26160==    by 0x334ACE767C: clone (in /lib64/libc-2.12.so)
==26160== 
[root@intel-w3520-8-2 ~]# ps aux|grep libvirtd
root     26160  3.4  1.7 536276 141892 ?       Sl   13:02   0:06 valgrind libvirtd -d
root     26286  0.0  0.0 103244   860 pts/2    S+   13:05   0:00 grep libvirtd
[root@intel-w3520-8-2 ~]# kill 26160
[root@intel-w3520-8-2 ~]# ==26160== 
==26160== HEAP SUMMARY:
==26160==     in use at exit: 956,598 bytes in 9,669 blocks
==26160==   total heap usage: 179,210 allocs, 169,541 frees, 451,276,713 bytes allocated
==26160== 
==26160== LEAK SUMMARY:
==26160==    definitely lost: 0 bytes in 0 blocks
==26160==    indirectly lost: 0 bytes in 0 blocks
==26160==      possibly lost: 4,048 bytes in 11 blocks
==26160==    still reachable: 952,550 bytes in 9,658 blocks
==26160==         suppressed: 0 bytes in 0 blocks
==26160== Rerun with --leak-check=full to see details of leaked memory
==26160== 
==26160== For counts of detected and suppressed errors, rerun with: -v
==26160== Use --track-origins=yes to see where uninitialised values come from
==26160== ERROR SUMMARY: 49 errors from 12 contexts (suppressed: 29 from 9)



Expected results:
Fix memory leak and errors.


Additional info:
Comment 2 Alex Jia 2012-08-24 03:05:51 EDT
BTW, the bug 769517 is used for tracking libvirtd memory leaks on RHEL6.3.
Comment 3 Dave Allan 2012-10-08 10:19:43 EDT
These BZs with valgrind logs are really not all that useful as many of the valgrind messages do not reflect specific problems that need to be fixed and the logs become stale as soon as new builds are released.  I'm going to close this BZ, but if there are specific leaks that are present in the most recent builds, please open BZs for them.  We are also working to implement nightly valgrind and coverity runs so it should not be necessary to run static analysis tools manually.

Note You need to log in before you can comment on or make changes to this bug.