From: Peter Krempa <pkrempa@redhat.com>
Date: Tue, 2 Mar 2021 11:50:36 +0000 (+0100)
Subject: virFileLoopDeviceAssociate: Use virStrcpy instead of virStrncpy
X-Git-Tag: v7.2.0-rc1~231
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9595c6162589289eadb53c5ef9e5ee55b232e994;p=thirdparty%2Flibvirt.git

virFileLoopDeviceAssociate: Use virStrcpy instead of virStrncpy

Passing 'strlen(src)' for length makes it equivalent to virStrcpy.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/src/util/virfile.c b/src/util/virfile.c
index d1da619666..180f2af405 100644
--- a/src/util/virfile.c
+++ b/src/util/virfile.c
@@ -810,8 +810,7 @@ int virFileLoopDeviceAssociate(const char *file,
     lo.lo_flags = LO_FLAGS_AUTOCLEAR;
 
     /* Set backing file name for LOOP_GET_STATUS64 queries */
-    if (virStrncpy((char *) lo.lo_file_name, file,
-                   strlen(file), LO_NAME_SIZE) < 0) {
+    if (virStrcpy((char *) lo.lo_file_name, file, LO_NAME_SIZE) < 0) {
         virReportSystemError(errno,
                              _("Unable to set backing file %s"), file);
         goto cleanup;