Bugfix: xenstat_collect_networks now returns the correct error code.

Signed-off-by: Josh Triplett <josht@us.ibm.com>
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

diff --git a/tools/xenstat/libxenstat/src/xenstat.c b/tools/xenstat/libxenstat/src/xenstat.c
index b5ce24d5cc1414f252849dcb9223c09e036731a3..d5a44961925d99b33a9acf558ea04e4492e9c548 100644 (file)
--- a/tools/xenstat/libxenstat/src/xenstat.c
+++ b/tools/xenstat/libxenstat/src/xenstat.c
@@ -493,20 +493,20 @@ static int xenstat_collect_networks(xenstat_node * node)
                node->handle->procnetdev = fopen("/proc/net/dev", "r");
                if (node->handle->procnetdev == NULL) {
                        perror("Error opening /proc/net/dev");
-                       return 1;
+                       return 0;
                }
 
                /* Validate the format of /proc/net/dev */
                if (fread(header, sizeof(PROCNETDEV_HEADER) - 1, 1,
                          node->handle->procnetdev) != 1) {
                        perror("Error reading /proc/net/dev header");
-                       return 1;
+                       return 0;
                }
                header[sizeof(PROCNETDEV_HEADER) - 1] = '\0';
                if (strcmp(header, PROCNETDEV_HEADER) != 0) {
                        fprintf(stderr,
                                "Unexpected /proc/net/dev format\n");
-                       return 1;
+                       return 0;
                }
        }
 
@@ -558,7 +558,7 @@ static int xenstat_collect_networks(xenstat_node * node)
                                    sizeof(xenstat_network));
                }
                if (domain->networks == NULL)
-                       return 1;
+                       return 0;
                domain->networks[domain->num_networks - 1] = net;
        }