Hide Forgot
Description of problem: The patch introduces a call of malloc(), but never calls free(). Version-Release number of selected component (if applicable): microcode_ctl-1.17-4.el6
Since this issue was entered in bugzilla, the release flag has been set to ? to ensure that it is properly evaluated for this release.
Created attachment 483660 [details] proposed fix
Created attachment 487268 [details] microcode_ctl.patch with a memleak fix included $ sudo valgrind /sbin/microcode_ctl -u ==5757== LEAK SUMMARY: ==5757== definitely lost: 8,000,000 bytes in 1 blocks $ sudo valgrind /sbin/microcode_ctl -u ==5824== HEAP SUMMARY: ==5824== in use at exit: 0 bytes in 0 blocks ==5824== total heap usage: 2 allocs, 2 frees, 8,000,568 bytes allocated ==5824== All heap blocks were freed -- no leaks are possible
Confirmed leak fixed in microcode_ctl-1.17-8.el6: # rpm -q microcode_ctl microcode_ctl-1.17-4.el6.x86_64 # sudo valgrind /sbin/microcode_ctl -u ... ==2742== LEAK SUMMARY: ==2742== definitely lost: 0 bytes in 0 blocks ==2742== indirectly lost: 0 bytes in 0 blocks ==2742== possibly lost: 8,000,000 bytes in 1 blocks ... # rpm -q microcode_ctl microcode_ctl-1.17-8.el6.x86_64 # sudo valgrind /sbin/microcode_ctl -u ... ==2858== HEAP SUMMARY: ==2858== in use at exit: 0 bytes in 0 blocks ==2858== total heap usage: 2 allocs, 2 frees, 8,000,568 bytes allocated ==2858== ==2858== All heap blocks were freed -- no leaks are possible ...
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHEA-2011-1594.html