util: Rename and move virStrIsPrint to virStringIsPrintable

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 48c7bc5c029eba24c7128ef61f778de3fa0f68ca..a3739e7df4114afe5d43389f4d6b348c2245645e 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -7160,7 +7160,7 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt,
                     goto error;
                 }
 
-                if (!virStrIsPrint(vendor)) {
+                if (!virStringIsPrintable(vendor)) {
                     virReportError(VIR_ERR_XML_ERROR, "%s",
                                    _("disk vendor is not printable string"));
                     goto error;
@@ -7175,7 +7175,7 @@ virDomainDiskDefParseXML(virDomainXMLOptionPtr xmlopt,
                     goto error;
                 }
 
-                if (!virStrIsPrint(product)) {
+                if (!virStringIsPrintable(product)) {
                     virReportError(VIR_ERR_XML_ERROR, "%s",
                                    _("disk product is not printable string"));
                     goto error;

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 068bc0026348e986a1d848a747f5a020277193f3..a79d85e77778d3945c0adabbef0b7792f86bb430 100644 (file)
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -2265,6 +2265,7 @@ virStringFreeListCount;
 virStringGetFirstWithPrefix;
 virStringHasControlChars;
 virStringIsEmpty;
+virStringIsPrintable;
 virStringJoin;
 virStringListLength;
 virStringReplace;
@@ -2478,7 +2479,6 @@ virSetNonBlock;
 virSetSockReuseAddr;
 virSetUIDGID;
 virSetUIDGIDWithCaps;
-virStrIsPrint;
 virTristateBoolTypeFromString;
 virTristateBoolTypeToString;
 virTristateSwitchTypeFromString;

diff --git a/src/util/virstring.c b/src/util/virstring.c
index 2d7fbf3eb4644b4f0f12447477b0f48e8025a11e..735e65ba1ef7dcfd868a62203b5e627df7792ecd 100644 (file)
--- a/src/util/virstring.c
+++ b/src/util/virstring.c
@@ -1048,3 +1048,21 @@ virStringToUpper(char **dst, const char *src)
     *dst = cap;
     return 1;
 }
+
+
+/**
+ * virStringIsPrintable:
+ *
+ * Returns true @str contains only printable characters.
+ */
+bool
+virStringIsPrintable(const char *str)
+{
+    size_t i;
+
+    for (i = 0; str[i]; i++)
+        if (!c_isprint(str[i]))
+            return false;
+
+    return true;
+}

diff --git a/src/util/virstring.h b/src/util/virstring.h
index 16ed3b29a50c2abd4d634607e4da7fb51918b254..fd2868a50f2a25893473c55f3de5483c0486c4d5 100644 (file)
--- a/src/util/virstring.h
+++ b/src/util/virstring.h
@@ -275,4 +275,6 @@ void virStringStripIPv6Brackets(char *str);
 bool virStringHasControlChars(const char *str);
 void virStringStripControlChars(char *str);
 
+bool virStringIsPrintable(const char *str);
+
 #endif /* __VIR_STRING_H__ */

diff --git a/src/util/virutil.c b/src/util/virutil.c
index f596d7289e1df6c35302b2176ae6e09e62e08edc..82051505d2585dd54434f95e4a71c0f1bc7d6b3b 100644 (file)
--- a/src/util/virutil.c
+++ b/src/util/virutil.c
@@ -1600,18 +1600,6 @@ virValidateWWN(const char *wwn)
     return true;
 }
 
-bool
-virStrIsPrint(const char *str)
-{
-    size_t i;
-
-    for (i = 0; str[i]; i++)
-        if (!c_isprint(str[i]))
-            return false;
-
-    return true;
-}
-
 #if defined(major) && defined(minor)
 int
 virGetDeviceID(const char *path, int *maj, int *min)

diff --git a/src/util/virutil.h b/src/util/virutil.h
index b121de09975f2a233f100682ee2db0e242efafe9..1e51a25f701b61315c09d3dbaf27d0f4bea98f59 100644 (file)
--- a/src/util/virutil.h
+++ b/src/util/virutil.h
@@ -152,8 +152,6 @@ bool virIsDevMapperDevice(const char *dev_name) ATTRIBUTE_NONNULL(1);
 
 bool virValidateWWN(const char *wwn);
 
-bool virStrIsPrint(const char *str);
-
 int virGetDeviceID(const char *path,
                    int *maj,
                    int *min);