[libvirt] [PATCH for v5.3.0 06/17] security: Don't skip label restore on file systems lacking XATTRs

[Cole Robinson]

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 <mprivozn at 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 <crobinso at redhat.com>

- Cole