Red Hat Bugzilla – Bug 228749
[RHEL4][patch] cid-17 cid-18 Fix memory leak in libe2p (e2p_edit_mntopts)
Last modified: 2018-10-19 18:58:50 EDT
Description of problem:
Another small memory leak in e2fsprogs. This time affecting the e2p_edit_mntopts
A temporary buffer is allocated and never freed upon return.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Use an e2fsprogs command that will edit mountopts, e.g. "tune2fs -ouid_16"
2. Run the command under valgrind, e.g.:
# valgrind --tool=memcheck tune2fs -ouid_16 /tmp/img0
A leak for this buffer is evident in --leak-check=yes output.
==22480== 7 bytes in 1 blocks are definitely lost in loss record 1 of 16
==22480== at 0x4004405: malloc (vg_replace_malloc.c:149)
==22480== by 0x402C82D: e2p_edit_mntopts (mntopts.c:102)
==22480== by 0x804A2AF: main (tune2fs.c:270)
==22480== LEAK SUMMARY:
==22480== definitely lost: 39 bytes in 2 blocks.
==22480== indirectly lost: 944 bytes in 50 blocks.
==22480== possibly lost: 0 bytes in 0 blocks.
==22480== still reachable: 12,022 bytes in 18 blocks.
==22480== suppressed: 0 bytes in 0 blocks.
==22480== Reachable blocks (those to which a pointer was found) are not shown.
==22480== To see them, rerun with: --show-reachable=yes
No leak for this buffer is evident in --leak-check=yes output.
==22682== LEAK SUMMARY:
==22682== definitely lost: 32 bytes in 1 blocks.
==22682== indirectly lost: 944 bytes in 50 blocks.
==22682== possibly lost: 0 bytes in 0 blocks.
==22682== still reachable: 12,022 bytes in 18 blocks.
==22682== suppressed: 0 bytes in 0 blocks.
==22682== Reachable blocks (those to which a pointer was found) are not shown.
==22682== To see them, rerun with: --show-reachable=yes
Created attachment 148083 [details]
Correctly free buffer before returning from e2p_edit_mntopts
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release. Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products. This request is not yet committed for inclusion in an Update
upstream commit for both patches http://thunk.org/hg/e2fsprogs/?rev/35af21f328c2
Applying a DEV_ACK for Eric Sandeen <email@example.com>. This bug
was found by coverity and has a patch available.
In cvs as of e2fsprogs-1.35-12.7.el4
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.