summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--usr.sbin/ypldap/aldap.c12
1 files changed, 7 insertions, 5 deletions
diff --git a/usr.sbin/ypldap/aldap.c b/usr.sbin/ypldap/aldap.c
index 84bf03af9f1..e459b3cb165 100644
--- a/usr.sbin/ypldap/aldap.c
+++ b/usr.sbin/ypldap/aldap.c
@@ -1,5 +1,5 @@
-/* $Id: aldap.c,v 1.8 2008/12/27 19:46:12 aschrijver Exp $ */
-/* $OpenBSD: aldap.c,v 1.8 2008/12/27 19:46:12 aschrijver Exp $ */
+/* $Id: aldap.c,v 1.9 2008/12/28 22:03:59 blambert Exp $ */
+/* $OpenBSD: aldap.c,v 1.9 2008/12/28 22:03:59 blambert Exp $ */
/*
* Copyright (c) 2008 Alexander Schrijver <aschrijver@openbsd.org>
@@ -141,7 +141,7 @@ aldap_parse(struct aldap *ldap)
return NULL;
if((m->msg = ber_read_elements(&ldap->ber, NULL)) == NULL)
- return NULL;
+ goto parsefail;
LDAP_DEBUG("message", m->msg);
@@ -168,7 +168,7 @@ aldap_parse(struct aldap *ldap)
break;
case LDAP_RES_SEARCH_ENTRY:
if(ber_scanf_elements(m->protocol_op, "{eS{e", &m->dn,
- &m->body.search.entries) != 0)
+ &m->body.search.entries) != 0)
goto parsefail;
break;
case LDAP_RES_SEARCH_REFERENCE:
@@ -179,13 +179,15 @@ aldap_parse(struct aldap *ldap)
return m;
parsefail:
+ aldap_freemsg(m);
return NULL;
}
void
aldap_freemsg(struct aldap_message *msg)
{
- ber_free_elements(msg->msg);
+ if (msg->msg)
+ ber_free_elements(msg->msg);
free(msg);
}