Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/lib/libkvm



Andrew Doran wrote:

> Thanks for fixing this. Redefining kauth_cred_t in userland isn't the right
> way to go about it though. Please change it so that kauth_cred_t is defined
> in a non-installed header file under sys/sys.

I'd rather not. Having it exposed in sys/sys is just asking for people
to access its members directly. Since libkvm must always be synchronized
with the kernel when data-structures it cares about are modified, having
a private copy in libkvm with the relevant members (note the lack of the
specificdata reference) seems saner to me.

-e.



Home | Main Index | Thread Index | Old Index