|
Ian Kent |
cc58c1 |
autofs-5.0.6 - fix fix LDAP result leaks on error paths
|
|
Ian Kent |
cc58c1 |
|
|
Ian Kent |
cc58c1 |
From: Ian Kent <raven@themaw.net>
|
|
Ian Kent |
cc58c1 |
|
|
Ian Kent |
cc58c1 |
The previous patch with which ensured that the result struture returned
|
|
Ian Kent |
cc58c1 |
from ldap_search_s(3) was freed could sometimes lead to a segmentation
|
|
Ian Kent |
cc58c1 |
fault because the local variable used was not initialized before use.
|
|
Ian Kent |
cc58c1 |
---
|
|
Ian Kent |
cc58c1 |
|
|
Ian Kent |
cc58c1 |
modules/lookup_ldap.c | 4 ++--
|
|
Ian Kent |
cc58c1 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
Ian Kent |
cc58c1 |
|
|
Ian Kent |
cc58c1 |
|
|
Ian Kent |
cc58c1 |
--- autofs-5.0.6.orig/modules/lookup_ldap.c
|
|
Ian Kent |
cc58c1 |
+++ autofs-5.0.6/modules/lookup_ldap.c
|
|
Ian Kent |
cc58c1 |
@@ -1521,7 +1521,7 @@ int lookup_read_master(struct master *ma
|
|
Ian Kent |
cc58c1 |
char buf[MAX_ERR_BUF];
|
|
Ian Kent |
cc58c1 |
char parse_buf[PARSE_MAX_BUF];
|
|
Ian Kent |
cc58c1 |
char *query;
|
|
Ian Kent |
cc58c1 |
- LDAPMessage *result, *e;
|
|
Ian Kent |
cc58c1 |
+ LDAPMessage *result = NULL, *e;
|
|
Ian Kent |
cc58c1 |
char *class, *info, *entry;
|
|
Ian Kent |
cc58c1 |
char **keyValue = NULL;
|
|
Ian Kent |
cc58c1 |
char **values = NULL;
|
|
Ian Kent |
cc58c1 |
@@ -2467,7 +2467,7 @@ static int lookup_one(struct autofs_poin
|
|
Ian Kent |
cc58c1 |
char buf[MAX_ERR_BUF];
|
|
Ian Kent |
cc58c1 |
time_t age = time(NULL);
|
|
Ian Kent |
cc58c1 |
char *query;
|
|
Ian Kent |
cc58c1 |
- LDAPMessage *result, *e;
|
|
Ian Kent |
cc58c1 |
+ LDAPMessage *result = NULL, *e;
|
|
Ian Kent |
cc58c1 |
char *class, *info, *entry;
|
|
Ian Kent |
cc58c1 |
char *enc_key1, *enc_key2;
|
|
Ian Kent |
cc58c1 |
int enc_len1 = 0, enc_len2 = 0;
|