[PATCH 7/8] virSecurityDACSetOwnershipInternal: Fix WIN32 code

Michal Privoznik mprivozn at redhat.com
Thu Jun 17 10:42:07 UTC 2021


I must admit, I have no idea why we build such POSIX dependent
code as DAC driver for something such not POSIX as WIN32. Anyway,
the code which is supposed to set error is not doing that. The
proper way is to mimic what chown() does:

  On error, -1 is returned, and errno is set to indicate the error.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/security/security_dac.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/security/security_dac.c b/src/security/security_dac.c
index 603d5b98ef..7ba367755a 100644
--- a/src/security/security_dac.c
+++ b/src/security/security_dac.c
@@ -700,7 +700,8 @@ virSecurityDACSetOwnershipInternal(const virSecurityDACData *priv,
         }
 
 #ifdef WIN32
-        rc = ENOSYS;
+        rc = -1;
+        errno = ENOSYS;
 #else /* !WIN32 */
         rc = chown(path, uid, gid);
 #endif /* !WIN32 */
-- 
2.31.1




More information about the libvir-list mailing list