Date: Tue, 19 Dec 1995 20:12:15 -0500
From: Theodore Ts'o <tytso@MIT.EDU>
Message-Id: <9512200112.AA23685@dcl.MIT.EDU>
To: Mark Carson <carson@cs.umd.edu>
Cc: krb5-bugs@MIT.EDU
In-Reply-To: "[1743] in Kerberos-V5-bugs"
Subject: Re: Bug fixes for May beta 5 code
Address: 1 Amherst St., Cambridge, MA 02139
Phone: (617) 253-8091
Content-Length: 1112

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