mirror of
https://git.proxmox.com/git/qemu
synced 2025-08-15 01:01:11 +00:00
virtio-9p: removexattr on default acl should return 0
If we don't have default acl, removexattr on default acl should return 0 Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> Signed-off-by: Venkateswararao Jujjuri <jvrao@linux.vnet.ibm.com>
This commit is contained in:
parent
3979251518
commit
a09947617c
@ -60,7 +60,7 @@ static int mp_pacl_removexattr(FsContext *ctx,
|
|||||||
ret = lremovexattr(rpath(ctx, path), MAP_ACL_ACCESS);
|
ret = lremovexattr(rpath(ctx, path), MAP_ACL_ACCESS);
|
||||||
if (ret == -1 && errno == ENODATA) {
|
if (ret == -1 && errno == ENODATA) {
|
||||||
/*
|
/*
|
||||||
* We don't get ENODATA error when trying to remote a
|
* We don't get ENODATA error when trying to remove a
|
||||||
* posix acl that is not present. So don't throw the error
|
* posix acl that is not present. So don't throw the error
|
||||||
* even in case of mapped security model
|
* even in case of mapped security model
|
||||||
*/
|
*/
|
||||||
@ -103,7 +103,18 @@ static int mp_dacl_setxattr(FsContext *ctx, const char *path, const char *name,
|
|||||||
static int mp_dacl_removexattr(FsContext *ctx,
|
static int mp_dacl_removexattr(FsContext *ctx,
|
||||||
const char *path, const char *name)
|
const char *path, const char *name)
|
||||||
{
|
{
|
||||||
return lremovexattr(rpath(ctx, path), MAP_ACL_DEFAULT);
|
int ret;
|
||||||
|
ret = lremovexattr(rpath(ctx, path), MAP_ACL_DEFAULT);
|
||||||
|
if (ret == -1 && errno == ENODATA) {
|
||||||
|
/*
|
||||||
|
* We don't get ENODATA error when trying to remove a
|
||||||
|
* posix acl that is not present. So don't throw the error
|
||||||
|
* even in case of mapped security model
|
||||||
|
*/
|
||||||
|
errno = 0;
|
||||||
|
ret = 0;
|
||||||
|
}
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user