[1748] in Kerberos-V5-bugs
Re: Bug fixes for May beta 5 code
daemon@ATHENA.MIT.EDU (Theodore Ts'o)
Tue Dec 19 20:12:30 1995
Date: Tue, 19 Dec 1995 20:12:15 -0500
From: Theodore Ts'o <tytso@MIT.EDU>
To: Mark Carson <carson@cs.umd.edu>
Cc: krb5-bugs@MIT.EDU
In-Reply-To: "[1743] in Kerberos-V5-bugs"
Thanks for your bug reports. A number of your reported bugs have
already been fixed in our release. I'll make comments on those that
weren't.
1. src/lib/krb4/kparse.c -- two variables are declared both static and
(in the header file) extern. I assumed they were meant to be extern.
Actually, the references to those variables in the header files should
have been removed, and that's what I've done in our release. (The two
varaibles are used anywhere, and they violate the namepsace
conventions.)
7. src/lib/krb5/krb/rd_cred.c: line 132 -- missing parentheses, so
wrong length was allocated.
Thanks for catching this!
8. src/lib/krb5/krb/rd_cred.c: line 173 -- the forwarded ticket was
being freed immediately after being received and parsed, but before
being stored. I simply dropped the free call, but presumably it really
should be moved elsewhere.
The correct fix (which we had in our tree already) is to change the
krb5_free_data() call to krb5_xfree(). This frees the krb5_data
container, but not the data portion of it, which is used by another
structure.
- Ted
