[PATCHv2 2/2] sysinfo: convert to virCommand

[Eric Blake <eblake@xxxxxxxxxx>]

* src/util/sysinfo.c (virSysinfoRead): Use virCommand instead of
virExec.
---
v2: remove unused variable, rebase on top of whitespace cleanups.

This patch is awaiting review on these prerequisites:
https://www.redhat.com/archives/libvir-list/2010-December/msg00318.html
https://www.redhat.com/archives/libvir-list/2010-December/msg00321.html

 src/util/sysinfo.c |   45 ++++++---------------------------------------
 1 files changed, 6 insertions(+), 39 deletions(-)

diff --git a/src/util/sysinfo.c b/src/util/sysinfo.c
index a02bd57..5b40a10 100644
--- a/src/util/sysinfo.c
+++ b/src/util/sysinfo.c
@@ -36,6 +36,7 @@
 #include "conf/domain_conf.h"
 #include "logging.h"
 #include "memory.h"
+#include "command.h"

 #define VIR_FROM_THIS VIR_FROM_SYSINFO

@@ -94,12 +95,9 @@ virSysinfoRead(void) {
 virSysinfoDefPtr
 virSysinfoRead(void) {
     char *path, *cur, *eol, *base;
-    int pid, outfd = -1, errfd = -1;
     virSysinfoDefPtr ret = NULL;
-    const char *argv[] = { NULL, "-q", "-t", "0,1", NULL };
-    int res, waitret, exitstatus;
     char *outbuf = NULL;
-    char *errbuf = NULL;
+    virCommandPtr cmd;

     path = virFindFileInPath(SYSINFO_SMBIOS_DECODER);
     if (path == NULL) {
@@ -108,43 +106,13 @@ virSysinfoRead(void) {
                              SYSINFO_SMBIOS_DECODER);
         return NULL;
     }
-    argv[0] = path;

-    res = virExec(argv, NULL, NULL, &pid, -1, &outfd, &errfd,
-                  VIR_EXEC_NONE | VIR_EXEC_NONBLOCK);
-    if (res < 0) {
+    cmd = virCommandNewArgList(path, "-q", "-t", "0,1", NULL);
+    virCommandSetOutputBuffer(cmd, &outbuf);
+    if (virCommandRun(cmd, NULL) < 0) {
         virSmbiosReportError(VIR_ERR_INTERNAL_ERROR,
                              _("Failed to execute command %s"),
                              path);
-        res = 1;
-        goto cleanup;
-    }
-
-    /*
-     * we are interested in the output, capture it and errors too
-     */
-    if (virPipeReadUntilEOF(outfd, errfd, &outbuf, &errbuf) < 0) {
-        virReportSystemError(errno, _("cannot wait for '%s'"), path);
-        while (waitpid(pid, &exitstatus, 0) == -1 && errno == EINTR)
-            ;
-        goto cleanup;
-    }
-
-    if (outbuf)
-        VIR_DEBUG("Command stdout: %s", outbuf);
-    if (errbuf)
-        VIR_DEBUG("Command stderr: %s", errbuf);
-
-    while ((waitret = waitpid(pid, &exitstatus, 0) == -1) &&
-           (errno == EINTR));
-    if (waitret == -1) {
-        virReportSystemError(errno, _("Failed to wait for '%s'"), path);
-        goto cleanup;
-    }
-    if (exitstatus != 0) {
-        virSmbiosReportError(VIR_ERR_INTERNAL_ERROR,
-                             _("command %s failed with error code %d:%s"),
-                             path, exitstatus, errbuf);
         goto cleanup;
     }

@@ -227,8 +195,7 @@ virSysinfoRead(void) {

 cleanup:
     VIR_FREE(outbuf);
-    VIR_FREE(errbuf);
-    VIR_FREE(path);
+    virCommandFree(cmd);

     return ret;

-- 
1.7.3.2

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list