Just a small change that was proposed in another thread.
From c57cff5606233bac109d2582878781b4e1ecb908 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <[email protected]>
Date: Wed, 19 Dec 2012 12:04:48 +0100
Subject: [PATCH] use talloc_zfree when freeing rhostent in resolver
We should use talloc_zfree() when freeing state variables, so we
can later avoid undesirable access after free.
---
src/resolv/async_resolv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/resolv/async_resolv.c b/src/resolv/async_resolv.c
index 268d266a43fec8bfb20ddb09aa75a893c8b9de9c..0465151184ee81694b247be0088e849d5e4d0795 100644
--- a/src/resolv/async_resolv.c
+++ b/src/resolv/async_resolv.c
@@ -1051,7 +1051,7 @@ resolv_gethostbyname_dns_parse(struct gethostbyname_dns_state *state,
* c-ares, but we need to handle it gracefully.
*/
if (state->rhostent->addr_list == NULL) {
- talloc_free(state->rhostent);
+ talloc_zfree(state->rhostent);
return ENOENT;
}
}
--
1.7.11.7
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
