summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorTheodore Tso <tytso@mit.edu>1994-07-16 03:39:20 +0000
committerTheodore Tso <tytso@mit.edu>1994-07-16 03:39:20 +0000
commitd9b286523a8c470c42eeb532799179ba0b97861a (patch)
tree9db279f7e29aabce31bed4ac779f4c1292d51b79 /src
parente0e0fc3d6654715013800aaa240b83011580bd60 (diff)
downloadkrb5-d9b286523a8c470c42eeb532799179ba0b97861a.tar.gz
krb5-d9b286523a8c470c42eeb532799179ba0b97861a.tar.xz
krb5-d9b286523a8c470c42eeb532799179ba0b97861a.zip
Fix core dump caused by trying to free stack garbage. :-)
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@3984 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src')
-rw-r--r--src/appl/telnet/libtelnet/ChangeLog6
-rw-r--r--src/appl/telnet/libtelnet/kerberos5.c4
2 files changed, 8 insertions, 2 deletions
diff --git a/src/appl/telnet/libtelnet/ChangeLog b/src/appl/telnet/libtelnet/ChangeLog
new file mode 100644
index 000000000..27a5c412a
--- /dev/null
+++ b/src/appl/telnet/libtelnet/ChangeLog
@@ -0,0 +1,6 @@
+Fri Jul 15 23:36:50 1994 Theodore Y. Ts'o (tytso at tsx-11)
+
+ * kerberos5.c (kerberos5_is): Avoid coredump caused by freeing of
+ an unitialized variable. Also make sure we don't try to free name
+ if it is NULL.
+
diff --git a/src/appl/telnet/libtelnet/kerberos5.c b/src/appl/telnet/libtelnet/kerberos5.c
index 0bc45cc70..2fd745ad8 100644
--- a/src/appl/telnet/libtelnet/kerberos5.c
+++ b/src/appl/telnet/libtelnet/kerberos5.c
@@ -354,7 +354,6 @@ kerberos5_is(ap, data, cnt)
printf("%s\r\n", errbuf);
return;
}
- free(name);
if ((ap->way & AUTH_HOW_MASK) == AUTH_HOW_MUTUAL) {
/* do ap_rep stuff here */
reply.ctime = authdat->authenticator->ctime;
@@ -382,7 +381,8 @@ kerberos5_is(ap, data, cnt)
}
auth_finished(ap, AUTH_USER);
- free(name);
+ if (name)
+ free(name);
if (authdat->authenticator->subkey &&
authdat->authenticator->subkey->keytype == KEYTYPE_DES) {
if (session_key.contents)