On 3/28/19 11:04 AM, Michal Privoznik wrote:
> The way that virSecurityDACRecallLabel is currently written is
> that if XATTRs are not supported for given path to the caller
> this is not different than if the path is still in use. The value
> of 1 is returned which makes secdrivers skip label restore.
> This is clearly a bug as we are not restoring labels on say NFS
> even though previously we were.
> 
> Strictly speaking, changes to virSecurityDACRememberLabel are not
> needed, but they are done anyway so that getter and setter behave
> in the same fashion.
> 
> Signed-off-by: Michal Privoznik <mpriv...@redhat.com>
> ---
>  src/security/security_dac.c     | 18 ++++++++++++------
>  src/security/security_selinux.c | 21 +++++++++++++++------
>  src/security/security_util.c    |  6 ++++--
>  3 files changed, 31 insertions(+), 14 deletions(-)

Reviewed-by: Cole Robinson <crobi...@redhat.com>

- Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to