[Fedora-directory-commits] ldapserver/ldap/servers/slapd modify.c, 1.12, 1.13

[Richard Allen Megginson (rmeggins)]

Author: rmeggins

Update of /cvs/dirsec/ldapserver/ldap/servers/slapd
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv16406

Modified Files:
	modify.c 
Log Message:
Bug(s) fixed: 185780
Bug Description: one byte memory leak in modify
Reviewed by: nhosoi (Thanks!)
Files: see diff
Branch: HEAD
Fix Description: Just call slapi_ch_free_string() with the mod->mod_type.  This is safe to call with NULL.
Platforms tested: RHEL4
Flag Day: no
Doc impact: no



Index: modify.c
===================================================================
RCS file: /cvs/dirsec/ldapserver/ldap/servers/slapd/modify.c,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -r1.12 -r1.13
--- modify.c	31 Aug 2006 22:52:07 -0000	1.12
+++ modify.c	12 Oct 2006 22:30:32 -0000	1.13
@@ -202,6 +202,7 @@
 			send_ldap_result( pb, LDAP_INVALID_SYNTAX, NULL, ebuf, 0, NULL );
 			slapi_ch_free((void **)&type);
 			ber_bvecfree(mod->mod_bvalues);
+			slapi_ch_free_string(&mod->mod_type);
 			slapi_ch_free((void **)&mod);
 			goto free_and_return;
 		}