[3756] in Kerberos-V5-bugs

home help back first fref pref prev next nref lref last post

[krbdev.mit.edu #1237] Merge the profile library with meeroh optimizations

daemon@ATHENA.MIT.EDU (Ken Raeburn via RT)
Fri Jan 10 21:42:12 2003

Mail-Followup-To: rt@krbdev.mit.edu
Message-Id: <rt-1237-3887.16.4392335068649@krbdev.mit.edu>
In-Reply-To: <rt-1237@krbdev.mit.edu>
From: "Ken Raeburn via RT" <rt-comment@krbdev.mit.edu>
Reply-To: rt-comment@krbdev.mit.edu
Mail-Copies-To: never
To: mjv@mit.edu
Cc: krb5-prs@mit.edu
Errors-To: krb5-bugs-admin@mit.edu
Date: Fri, 10 Jan 2003 21:41:14 -0500 (EST)

Yes, the locking model needs fixing.

profile_update_file_data can free and then create data->root, but
data->root isn't protected by the mutex, so another thread using a
different prf_file_t handle may be reading it while it's being freed.

I suspect the right fix is going to involve changing the keying of the
linked list to be by timestamp as well as by name.  Need to think
through how that's going to affect the update_file part.

For now, what's there should work just fine as long as the config file
isn't modified while a process is using it in multiple threads.
_______________________________________________
krb5-bugs mailing list
krb5-bugs@mit.edu
http://mailman.mit.edu/mailman/listinfo/krb5-bugs

home help back first fref pref prev next nref lref last post