[PATCH 2/2] auvirt: Remove workaround for VM name searching
Steve Grubb
sgrubb at redhat.com
Thu Feb 9 13:35:06 UTC 2012
On Thursday, February 09, 2012 08:22:34 AM Marcelo Cerri wrote:
> Thanks for your explanation. I hadn't notice how escaped fields work.
>
> Regarding the search algorithm fix, sorry but it is not clear to me
> where you meant to say to add the type check and the escape. Did you
> mean inside the ausearch_add_item or in the function which is calling
> the ausearch_add_item function?
I think its best to put it inside the function so that app writers do not have
to think about it. They just pass a string and its fixed up. I was also thinking
about the alternative, which is to decode the fields during search and then
compare. But this would be slower because we decode every field value whether it
matches or not. So, we can just encode the item being searched for and then
compare raw values. I suppose the man page should clarify this for app writers
just in case.
> I'll submit a patch to libvirt instead and then update auvirt.
I wished I caught that sooner, too. As for auvirt, since you know vm is an
escaped field, you don't actually need to put the "if" statement to check its
type. You can just call the interpret function unconditionally and use its
output.
Thanks,
-Steve
More information about the Linux-audit
mailing list