diff options
author | Panu Matilainen <pmatilai@redhat.com> | 2018-07-30 17:01:02 +0300 |
---|---|---|
committer | Panu Matilainen <pmatilai@redhat.com> | 2018-07-30 17:01:02 +0300 |
commit | 27c5ee3e4d8ccf17936ffb91a858676724f5cb5d (patch) | |
tree | e8258777469cc57ff2221a3b7ac9f1210a40efef /lib/verify.c | |
parent | fc449e06e5e7c311273864ca396f0c249eea9ecc (diff) | |
download | rpm-27c5ee3e4d8ccf17936ffb91a858676724f5cb5d.tar.gz |
Fix a memleak in file verification
Introduced in commit 591f95a849776aba2a6a9e42fbdd27754a2818b0: unlike
rpmfiFN(), rpmfilesFN() returns alloced data and thus needs to be freed.
Diffstat (limited to 'lib/verify.c')
-rw-r--r-- | lib/verify.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/verify.c b/lib/verify.c index d89996dea..97b4dfc13 100644 --- a/lib/verify.c +++ b/lib/verify.c @@ -57,7 +57,7 @@ rpmVerifyAttrs rpmfilesVerify(rpmfiles fi, int ix, rpmVerifyAttrs omitMask) { rpmfileAttrs fileAttrs = rpmfilesFFlags(fi, ix); rpmVerifyAttrs flags = rpmfilesVFlags(fi, ix); - const char * fn = rpmfilesFN(fi, ix); + char * fn = rpmfilesFN(fi, ix); struct stat sb, fsb; rpmVerifyAttrs vfy = RPMVERIFY_NONE; @@ -239,6 +239,7 @@ rpmVerifyAttrs rpmfilesVerify(rpmfiles fi, int ix, rpmVerifyAttrs omitMask) vfy |= RPMVERIFY_GROUP; exit: + free(fn); return vfy; } |