From: Matthias Bolte <matthias.bolte@googlemail.com>
Date: Tue, 30 Mar 2010 14:52:42 +0000 (+0200)
Subject: xen: Use virStrToLong_i instead of sscanf for XenD port parsing
X-Git-Tag: v0.8.0~124
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=57dab748049d9f815b58dcf781bdb5ff3c8d1d2f;p=thirdparty%2Flibvirt.git

xen: Use virStrToLong_i instead of sscanf for XenD port parsing

Parsing is stricter now and doesn't accept trailing characters
after the actual value anymore.
---

diff --git a/src/xen/xend_internal.c b/src/xen/xend_internal.c
index 0649d23a37..691a940bd3 100644
--- a/src/xen/xend_internal.c
+++ b/src/xen/xend_internal.c
@@ -4683,7 +4683,7 @@ xenDaemonDomainMigratePerform (virDomainPtr domain,
     else if ((p = strrchr (uri, ':')) != NULL) { /* "hostname:port" */
         int port_nr, n;
 
-        if (sscanf (p+1, "%d", &port_nr) != 1) {
+        if (virStrToLong_i(p+1, NULL, 10, &port_nr) < 0) {
             virXendError (conn, VIR_ERR_INVALID_ARG,
                           "%s", _("xenDaemonDomainMigrate: invalid port number"));
             return -1;