Skip Menu |
 

History Show all quoted text — Show full headers
# Wed Jun 20 10:59:35 2012 "Kevin Wasserman" <krwasserman@hotmail.com> - Ticket created
From: "Kevin Wasserman" <krwasserman@hotmail.com>
To: <krb5-bugs@mit.edu>
Subject: krb5_cc_get_full_name() does not document how to free fullname_out
Date: Wed, 20 Jun 2012 10:42:01 -0400
Download (untitled) / with headers
text/plain 201B
Download (untitled) / with headers
text/html 431B
Presumably it should be via krb5_free_string().  At least, that is how
kdestroy.c does it.  Note that kinit.c simply calls free(), but that is
presumably a bug.
 
Kevin Wasserman
Painless Security, LLC
# Wed Jun 20 11:06:44 2012 ghudson@mit.edu (Greg Hudson) - Correspondence added
Download (untitled) / with headers
text/plain 179B
It's not really a bug for klist (not kinit) to use free() for that, since
it should be built in the same way as libkrb5. But it doesn't set a very
good example, so I'll fix it.
# Wed Jun 20 11:14:33 2012 ghudson@mit.edu (Greg Hudson) - Given to ghudson@mit.edu (Greg Hudson)
# Wed Jun 20 11:14:33 2012 ghudson@mit.edu (Greg Hudson) - Status changed from 'new' to 'review'
# Wed Jun 20 11:14:33 2012 ghudson@mit.edu (Greg Hudson) - Correspondence added
From: ghudson@mit.edu
Subject: SVN Commit
Download (untitled) / with headers
text/plain 504B

Document how to free krb5_cc_get_full_name result

Also, in klist, use the appropriate libkrb5 free functions for
krb5_cc_get_full_name and krb5_unparse_name_results. Reported by
Kevin Wasserman.

https://github.com/krb5/krb5/commit/37b09d45192bd9d958d047dbb829d10f892b1fda
Author: Greg Hudson <ghudson@mit.edu>
Commit: 37b09d45192bd9d958d047dbb829d10f892b1fda
Branch: master
src/clients/klist/klist.c | 4 ++--
src/include/krb5/krb5.hin | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
# Mon Nov 19 16:09:50 2012 tlyu (Taylor Yu) - Version_Fixed 1.11 added
# Mon Nov 19 16:12:39 2012 tlyu (Taylor Yu) - Status changed from 'review' to 'resolved'