From: Tom Yu <tlyu@mit.edu>
Date: Thu, 22 Sep 2005 17:30:45 +0000 (+0000)
Subject: pullup r17388 from trunk
X-Git-Tag: krb5-1.4.3-beta1~18
X-Git-Url: http://git.tremily.us/gitweb.cgi?a=commitdiff_plain;h=304cb890e829a76b201eb0cb20b57703de7ef844;p=krb5.git

pullup r17388 from trunk

ticket: 3186
version_fixed: 1.4.3

git-svn-id: svn://anonsvn.mit.edu/krb5/branches/krb5-1-4@17394 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/lib/gssapi/generic/ChangeLog b/src/lib/gssapi/generic/ChangeLog
index 4480d3158..901410c17 100644
--- a/src/lib/gssapi/generic/ChangeLog
+++ b/src/lib/gssapi/generic/ChangeLog
@@ -1,3 +1,8 @@
+2005-09-22  Tom Yu  <tlyu@mit.edu>
+
+	* rel_buffer.c (generic_gss_release_buffer): Free buffer even if
+	length is zero.  This avoids memory leaks in some cases.
+
 2005-04-07  Tom Yu  <tlyu@mit.edu>
 
 	* Makefile.in (gssapi.h): Use awk hack to work around quoting
diff --git a/src/lib/gssapi/generic/rel_buffer.c b/src/lib/gssapi/generic/rel_buffer.c
index f5d923b09..555888184 100644
--- a/src/lib/gssapi/generic/rel_buffer.c
+++ b/src/lib/gssapi/generic/rel_buffer.c
@@ -47,8 +47,7 @@ generic_gss_release_buffer (minor_status,
     if (buffer == GSS_C_NO_BUFFER)
 	return(GSS_S_COMPLETE);
 
-    if ((buffer->length) &&
-	(buffer->value)) {
+    if (buffer->value) {
 	free(buffer->value);
 	buffer->length = 0;
 	buffer->value = NULL;