|
 |
0af72f |
From c6f2737747cbb70adfdd1a77412b669838f9c419 Mon Sep 17 00:00:00 2001
|
|
|
0af72f |
From: Rob Crittenden <rcritten@redhat.com>
|
|
|
0af72f |
Date: Mon, 2 Dec 2019 15:08:54 -0500
|
|
|
0af72f |
Subject: [PATCH] Fix use-after-free issue
|
|
|
0af72f |
|
|
|
0af72f |
The basedn value was freed after the first search but a second
|
|
|
0af72f |
one could be initiated.
|
|
|
0af72f |
---
|
|
|
0af72f |
src/ipa.c | 2 +-
|
|
|
0af72f |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
0af72f |
|
|
|
0af72f |
diff --git a/src/ipa.c b/src/ipa.c
|
|
|
0af72f |
index 40a4b52c..41ca9081 100644
|
|
|
0af72f |
--- a/src/ipa.c
|
|
|
0af72f |
+++ b/src/ipa.c
|
|
|
0af72f |
@@ -540,7 +540,6 @@ fetch_roots(const char *server, int ldap_uri_cmd, const char *ldap_uri,
|
|
|
0af72f |
/* Now look up the root certificates for the domain. */
|
|
|
0af72f |
snprintf(lfilter, sizeof(lfilter), "(%s=*)", lattrs[0]);
|
|
|
0af72f |
snprintf(ldn, sizeof(ldn), "%s,%s", relativedn, basedn);
|
|
|
0af72f |
- free(basedn);
|
|
|
0af72f |
rc = ldap_search_ext_s(ld, ldn, LDAP_SCOPE_SUBTREE,
|
|
|
0af72f |
lfilter, lattrs, 0, NULL, NULL, NULL,
|
|
|
0af72f |
LDAP_NO_LIMIT, &lresult);
|
|
|
0af72f |
@@ -551,6 +550,7 @@ fetch_roots(const char *server, int ldap_uri_cmd, const char *ldap_uri,
|
|
|
0af72f |
lfilter, lattrs, 0, NULL, NULL, NULL,
|
|
|
0af72f |
LDAP_NO_LIMIT, &lresult);
|
|
|
0af72f |
}
|
|
|
0af72f |
+ free(basedn);
|
|
|
0af72f |
if (rc != LDAP_SUCCESS) {
|
|
|
0af72f |
fprintf(stderr, "Error searching '%s': %s.\n",
|
|
|
0af72f |
ldn, ldap_err2string(rc));
|
|
|
0af72f |
--
|
|
|
0af72f |
2.21.0
|
|
|
0af72f |
|