On Thu, Mar 15, 2012 at 08:24:40PM +0100, Jakub Hrozek wrote:
I found this (well, valgrind did..) when debugging #1251. I
haven't found
any TDB specific free function, so I just used free..
From d2f9f81b334e6685c050f8b3e13a3a1266858a07 Mon Sep 17 00:00:00
2001
From: Jakub Hrozek <jhrozek(a)redhat.com>
Date: Wed, 14 Mar 2012 16:22:50 +0100
Subject: [PATCH] Free entry found in negative cache
---
src/responder/common/negcache.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/responder/common/negcache.c b/src/responder/common/negcache.c
index fab549c16d911985fbc37f112499e9f692103334..02a4d18023e5345363e5be997d78e2eebc5ba1a0
100644
--- a/src/responder/common/negcache.c
+++ b/src/responder/common/negcache.c
@@ -127,6 +127,7 @@ done:
ret = ENOENT;
}
+ free(data.dptr);
using free() is safe here, but you have to initialize data, or at least
data.dptr at the beginning of sss_ncache_check_str() because there is a
goto done before data is set.
bye,
Sumit
return ret;
}
--
1.7.7.6
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel