Re: [libvirt] [PATCH v3 25/28] security_dac: Fix info messages when chown()-ing

2018-08-31 Thread John Ferlan
On 08/27/2018 04:08 AM, Michal Privoznik wrote: > Firstly, the message that says we're setting uid:gid shouldn't be > called from virSecurityDACSetOwnershipInternal() because > virSecurityDACRestoreFileLabelInternal() is calling it too. > Secondly, there are places between us reporting label res

[libvirt] [PATCH v3 25/28] security_dac: Fix info messages when chown()-ing

2018-08-27 Thread Michal Privoznik
Firstly, the message that says we're setting uid:gid shouldn't be called from virSecurityDACSetOwnershipInternal() because virSecurityDACRestoreFileLabelInternal() is calling it too. Secondly, there are places between us reporting label restore and us actually doing it where we can quit. Don't say