[libvirt] [PATCH 4/6] audit: disk: Refactor disk auditing to avoid auditing remote storage

Peter Krempa pkrempa at redhat.com
Fri Jul 4 10:32:37 UTC 2014


On 07/03/14 15:30, Ján Tomko wrote:
> On 07/03/2014 12:05 PM, Peter Krempa wrote:
>> Pass the virStorageSource struct to the auditing function and check if
>> storage is local before auditting.
> 
> *auditing
> 
> This would be nice to document in auditlog.html

Well the auditlog doc states that we audit only *host resources*, thus
auditing (sometimes) incomplete info about a remote storage volume is
technically a bug.

> 
>> ---
>>  src/conf/domain_audit.c | 25 ++++++++++++++++---------
>>  src/conf/domain_audit.h |  4 ++--
>>  src/lxc/lxc_driver.c    |  6 +++---
>>  src/qemu/qemu_driver.c  |  4 ++--
>>  src/qemu/qemu_hotplug.c | 21 ++++++++-------------
>>  5 files changed, 31 insertions(+), 29 deletions(-)
>>
> 
> ACK
> 

Thanks, I'll push this one shortly.

Peter


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140704/2033816a/attachment-0001.sig>


More information about the libvir-list mailing list