[OpenAFS-devel] comment on applied kauth/user_nt.c patch

Joe Buehler jbuehler@hekimian.com
Tue, 02 Dec 2003 09:45:08 -0500


Looking at the current CVS, I see that the following snippet
in kauth/user_nt.c is still incorrect:

	if (krb_debug) {
	    fprintf(stderr, "select failed: readfds=%p", readfds);
	    perror("");
	}

I had submitted a patch containing a comment about the format
being incorrect.  I should have been less terse -- the problem is that
readfds may have a non-integral type.  It is a set of file descriptors,
so it might actually be an array or a struct.  The format was originally
%x, which will work for integer "readfds".  But for RedHat 9, for example,
readfds is actually a struct containing an array of 32 or so longs.
-- 
Joe Buehler