[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Using the kernel's audit.h



Hello,

Mounir brought to my attention that audit-0.6.5 now uses the kernel's
audit.h -- This is going to be a problem, currently and could be a
potential problem in the future.  I thought using kernel space headers
in user space programs was generally discouraged.

Currently, the auditfs piece uses kernel-specific structures.  If
using the kernel's audit.h is the way we're going to do it, though,
I'll attempt to hide my stuff with __KERNEL__, but I have a feeling
that using this macro is going to annoy people.  Then again, the audit
subsystem requires user space tools for it to be useful and this may
be permissible and less annoying.

Later, someone could add functionality to the audit subsystem without
changing old functionality and break the user space tools.  These
types of changes should not affect the user space tools, but if they
use kernel-specific structures without __KERNEL__, the audit tools
will not be compilable.  Of course we're hoping for someone to review
the patch and know that user space tools do use the kernel header
file....

Anyway, just voicing this concern.  I'd personally prefer just keeping
a copy of the header file in userspace for userspace.

-- 
- Timothy R. Chavez


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]