[krbdev.mit.edu #2858] CVS Commit

Ken Raeburn via RT rt-comment at krbdev.mit.edu
Mon Jan 3 14:30:25 EST 2005


As Ezra noted, krb5_parse_name saves away the default realm the first
time it looks it up, and will never look it up again, even if the
config file has changed, the context isn't the same one, etc.  It also
used a static variable which wasn't thread-safe.

With this patch, we still won't catch changes in the config file
specification after the default realm has been looked up for a given
krb5_context, because it's cached in the context too, but this will at
least be more responsive, and thread safe.

* parse.c (krb5_parse_name): Don't cache the default realm name.


To generate a diff of this commit:



	cvs diff -r5.455 -r5.456 krb5/src/lib/krb5/krb/ChangeLog
	cvs diff -r5.45 -r5.46 krb5/src/lib/krb5/krb/parse.c 


More information about the krb5-bugs mailing list