diff options
| author |   blambert <blambert@openbsd.org> | 2009-07-08 13:13:17 +0000 |
|---|---|---|
| committer | blambert <blambert@openbsd.org> | 2009-07-08 13:13:17 +0000 |
| commit | f62ce5816b3fea8ad08944b86d5557456ac47a41 (patch) | |
| tree | 9754898948936f2dfcec887ac5d7aedc7657c180 | |
| parent | When reinitialising the screen, do not omit to clear the last line. (diff) | |
| download | wireguard-openbsd-f62ce5816b3fea8ad08944b86d5557456ac47a41.tar.xz wireguard-openbsd-f62ce5816b3fea8ad08944b86d5557456ac47a41.zip | |
Fix memory leak in sarch filter parsing code.
ok pyr@
| -rw-r--r-- | usr.sbin/ypldap/aldap.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/usr.sbin/ypldap/aldap.c b/usr.sbin/ypldap/aldap.c index 2c9d47408af..70c096a477a 100644 --- a/usr.sbin/ypldap/aldap.c +++ b/usr.sbin/ypldap/aldap.c @@ -1,5 +1,5 @@ -/* $Id: aldap.c,v 1.19 2009/01/29 11:43:31 aschrijver Exp $ */ -/* $OpenBSD: aldap.c,v 1.19 2009/01/29 11:43:31 aschrijver Exp $ */ +/* $Id: aldap.c,v 1.20 2009/07/08 13:13:17 blambert Exp $ */ +/* $OpenBSD: aldap.c,v 1.20 2009/07/08 13:13:17 blambert Exp $ */ /* * Copyright (c) 2008 Alexander Schrijver <aschrijver@openbsd.org> @@ -781,11 +781,11 @@ ldap_do_parse_search_filter(struct ber_element *prev, char **cpp) if ((parsed_val = parseval(attr_val, len)) == NULL) goto callfail; - if ((elm = - ber_add_nstring(elm, parsed_val, - strlen(parsed_val))) == NULL) - goto callfail; + elm = ber_add_nstring(elm, parsed_val, + strlen(parsed_val)); free(parsed_val); + if (elm == NULL) + goto callfail; ber_set_header(elm, BER_CLASS_CONTEXT, type); if (type == LDAP_FILT_SUBS_FIN) break; @@ -795,10 +795,10 @@ ldap_do_parse_search_filter(struct ber_element *prev, char **cpp) if ((parsed_val = parseval(attr_val, len)) == NULL) goto callfail; - if ((elm = ber_add_nstring(elm, parsed_val, - strlen(parsed_val))) == NULL) - goto callfail; + elm = ber_add_nstring(elm, parsed_val, strlen(parsed_val)); free(parsed_val); + if (elm == NULL) + goto callfail; break; } |