From: Theodore Tso <tytso@mit.edu>
Date: Thu, 20 Aug 1992 01:30:33 +0000 (+0000)
Subject: Changed so that the krb5_data structures (in the principal structure)
X-Git-Tag: krb5-1.0-beta2~128
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=0be9bdba5d41c5530eda0ef9ef551c8d5309131b;p=krb5.git

Changed so that the krb5_data structures (in the principal structure)
have nulls at the end of the strings, just in case poeple are depending
on it.  They shouldn't, since principals component may have embedded
NULL's in them, even though they usually don't.  Nevertheless, it's good
to be careful.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@2334 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/lib/krb5/ccache/file/fcc_read.c b/src/lib/krb5/ccache/file/fcc_read.c
index e59e8c82a..b41ce1b9b 100644
--- a/src/lib/krb5/ccache/file/fcc_read.c
+++ b/src/lib/krb5/ccache/file/fcc_read.c
@@ -228,7 +228,7 @@ krb5_fcc_read_data(id, data)
 	return KRB5_OK;
      }
 
-     data->data = (char *) malloc(data->length);
+     data->data = (char *) malloc(data->length+1);
      if (data->data == NULL)
 	  return KRB5_CC_NOMEM;
 
@@ -242,11 +242,8 @@ krb5_fcc_read_data(id, data)
 	 xfree(data->data);
 	 return KRB5_CC_END;
      }
+     data->data[data->length] = 0; /* Null terminate, just in case.... */
      return KRB5_OK;
- errout:
-     if (data->data)
-	 xfree(data->data);
-     return kret;
 }
 
 krb5_error_code