lxc: do not require 'ifconfig' or 'ipconfig' in container

Currently, the lxc implementation invokes 'ip' and 'ifconfig' commands
inside a container using 'virRun'.  That has the side effect of requiring
those commands to be present and to function in a manner consistent with
the usage.  Some small roots (such as ttylinux) may not have 'ip' or
'ifconfig'.

This patch replaces the use of these commands with usage of
netdevice.  The result is that lxc containers do not have to implement
those commands, and lxc in libvirt is only dependent on the netdevice
interface.

I've tested this patch locally against the ubuntu libvirt version enough
to verify its generally sane.  I attempted to build upstream today, but
failed with:
  /usr/bin/ld:
    ../src/.libs/libvirt_driver_qemu.a(libvirt_driver_qemu_la-qemu_domain.o):
   undefined reference to symbol 'xmlXPathRegisterNs@@LIBXML2_2.4.30

Thats probably a local issue only, but I wanted to get this patch up and
see what others thought of it.  This is ubuntu bug
https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/828211 .

src/lxc/veth.c

@@ -1,7 +1,7 @@
 /*
  * veth.c: Tools for managing veth pairs
  *
- * Copyright (C) 2010 Red Hat, Inc.
+ * Copyright (C) 2010-2011 Red Hat, Inc.
  * Copyright IBM Corp. 2008
  *
  * See COPYING.LIB for the License of this software
@@ -12,8 +12,11 @@
 
 #include <config.h>
 
+#include <linux/sockios.h>
+#include <net/if.h>
 #include <string.h>
 #include <stdio.h>
+#include <sys/ioctl.h>
 #include <sys/types.h>
 #include <sys/wait.h>
 
@@ -23,6 +26,7 @@
 #include "memory.h"
 #include "command.h"
 #include "virterror_internal.h"
+#include "virfile.h"
 
 #define VIR_FROM_THIS VIR_FROM_LXC
 
@@ -186,41 +190,49 @@ int vethDelete(const char *veth)
  * @veth: name of veth device
  * @upOrDown: 0 => down, 1 => up
  *
- * Enables a veth device using the ifconfig command.  A NULL inetAddress
+ * Enables a veth device using SIOCSIFFLAGS
- * will cause it to be left off the command line.
  *
- * Returns 0 on success or -1 in case of error
+ * Returns 0 on success, -1 on failure, with errno set
  */
 int vethInterfaceUpOrDown(const char* veth, int upOrDown)
 {
-    int rc;
+    struct ifreq ifr;
-    const char *argv[] = {"ifconfig", veth, NULL, NULL};
+    int fd, ret;
-    int cmdResult = 0;
 
-    if (0 == upOrDown)
+    if ((fd = socket(PF_PACKET, SOCK_DGRAM, 0)) == -1)
-        argv[2] = "down";
+        return(-1);
+
+    memset(&ifr, 0, sizeof(struct ifreq));
+
+    if (virStrcpyStatic(ifr.ifr_name, veth) == NULL) {
+        errno = EINVAL;
+        return -1;
+    }
+
+    if ((ret = ioctl(fd, SIOCGIFFLAGS, &ifr)) == 0) {
+        if (upOrDown)
+            ifr.ifr_flags |= IFF_UP;
         else
-        argv[2] = "up";
+            ifr.ifr_flags &= ~(IFF_UP | IFF_RUNNING);
 
-    rc = virRun(argv, &cmdResult);
+        ret = ioctl(fd, SIOCSIFFLAGS, &ifr);
+    }
 
-    if (rc != 0 ||
+    VIR_FORCE_CLOSE(fd);
-        (WIFEXITED(cmdResult) && WEXITSTATUS(cmdResult) != 0)) {
+    if (ret == -1)
-        if (0 == upOrDown)
+        if (upOrDown == 0)
             /*
              * Prevent overwriting an error log which may be set
              * where an actual failure occurs.
              */
-            VIR_DEBUG("Failed to disable '%s' (%d)",
+            VIR_DEBUG("Failed to disable '%s'", veth);
-                      veth, WEXITSTATUS(cmdResult));
         else
             vethError(VIR_ERR_INTERNAL_ERROR,
-                      _("Failed to enable '%s' (%d)"),
+                      _("Failed to enable '%s'"), veth);
-                      veth, WEXITSTATUS(cmdResult));
+    else
-        rc = -1;
+        ret = 0;
-    }
 
-    return rc;
+    return(ret);
 }
 
 /**
@@ -279,17 +291,29 @@ int setMacAddr(const char* iface, const char* macaddr)
  * @iface: name of device
  * @new: new name of @iface
  *
- * Changes the name of the given device with the
+ * Changes the name of the given device.
- * given new name using this command:
- *     ip link set @iface name @new
  *
- * Returns 0 on success or -1 in case of error
+ * Returns 0 on success, -1 on failure with errno set.
  */
 int setInterfaceName(const char* iface, const char* new)
 {
-    const char *argv[] = {
+    struct ifreq ifr;
-        "ip", "link", "set", iface, "name", new, NULL
+    int fd = socket(PF_PACKET, SOCK_DGRAM, 0);
-    };
 
-    return virRun(argv, NULL);
+    memset(&ifr, 0, sizeof(struct ifreq));
+
+    if (virStrcpyStatic(ifr.ifr_name, iface) == NULL) {
+        errno = EINVAL;
+        return -1;
+    }
+
+    if (virStrcpyStatic(ifr.ifr_newname, new) == NULL) {
+        errno = EINVAL;
+        return -1;
+    }
+
+    if (ioctl(fd, SIOCSIFNAME, &ifr))
+        return -1;
+
+    return 0;
 }