External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00
Code

virFileSetXAttr: Report error on failure

Browse Source 
It's better to have the function report errors, because none of
the callers does.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Cole Robinson <crobinso@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Michal Privoznik 2019-03-22 15:34:59 +01:00
parent 0d44d2876a
commit 9b130c33f9
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/util/virfile.c
View File

@ -4421,14 +4421,21 @@ virFileGetXAttrQuiet(const char *path,
* Sets xattr of @name and @value on @path. * Sets xattr of @name and @value on @path.
* *
* Returns: 0 on success, * Returns: 0 on success,
* -1 otherwise (with errno set). * -1 otherwise (with errno set AND error reported).
*/ */
int int
virFileSetXAttr(const char *path, virFileSetXAttr(const char *path,
const char *name, const char *name,
const char *value) const char *value)
{ {
return setxattr(path, name, value, strlen(value), 0); if (setxattr(path, name, value, strlen(value), 0) < 0) {
virReportSystemError(errno,
_("Unable to set XATTR %s on %s"),
name, path);
return -1;
}
return 0;
} }
/** /**
@ -4460,11 +4467,14 @@ virFileGetXAttrQuiet(const char *path ATTRIBUTE_UNUSED,
} }
int int
virFileSetXAttr(const char *path ATTRIBUTE_UNUSED, virFileSetXAttr(const char *path,
const char *name ATTRIBUTE_UNUSED, const char *name,
const char *value ATTRIBUTE_UNUSED) const char *value ATTRIBUTE_UNUSED)
{ {
errno = ENOSYS; errno = ENOSYS;
virReportSystemError(errno,
_("Unable to set XATTR %s on %s"),
name, path);
return -1; return -1;
} }