Re: difficulty with TYPE
James Antill wrote:
On Tue, 2008-01-15 at 08:58 -0500, John Dennis wrote:
> James Antill wrote:
>> The second iovec above can't just be MAX_AUDIT_MESSAGE_LENGTH, or if
>> there are two messages you'll read some/all of the next one(s). You
>> either need to read the header first and then use hdr.size, or separate
>> the IO from the parsing.
>> Also you can't just check for readv() as above, you need to check that
>> you've read the amount of data you want, and if you didn't get it all
>> yet then loop.
> This is why we provide libraries to do things like this, it can be
> tricky to get right. The feed() interface to auparse consumes arbitrary
auparse_feed() works off log files and the audispd "string" format. The
above code was using the auditd -> audispd format, so that API doesn't
work.
Then it needs to be fixed to also work with the old binary protocol. But
on the other hand, the binary protocol is deprecated and won't be used
with the new audispd so perhaps it's moot.
FWIW, setroubleshoot supports automatic detection of the audispd
protocol and utilizes a feed interface so if backwards compatibility is
important for auparse we already have a proof of concept implementation
of how to do this.
--
John Dennis <jdennis(a)redhat.com>