[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen master] xenstat: don't leak memory in getBridge



commit 8a158bc59cfb4c51a355cefb0a6b565b83ad5028
Author:     Matthew Daley <mattd@xxxxxxxxxxx>
AuthorDate: Sun May 4 20:31:47 2014 +1200
Commit:     Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Thu May 8 10:48:15 2014 +0100

    xenstat: don't leak memory in getBridge
    
    getBridge's method of returning a result was a little confused:
    allocating a result buffer but never using it.
    
    Simplify by instead allowing a result buffer to be passed in and
    modifying the single usage to match.
    
    Signed-off-by: Matthew Daley <mattd@xxxxxxxxxxx>
    Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 tools/xenstat/libxenstat/src/xenstat_linux.c |   16 +++++++---------
 1 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/tools/xenstat/libxenstat/src/xenstat_linux.c 
b/tools/xenstat/libxenstat/src/xenstat_linux.c
index 931b24e..24335e1 100644
--- a/tools/xenstat/libxenstat/src/xenstat_linux.c
+++ b/tools/xenstat/libxenstat/src/xenstat_linux.c
@@ -64,14 +64,12 @@ static const char PROCNETDEV_HEADER[] =
 
 /* We need to get the name of the bridge interface for use with bonding 
interfaces */
 /* Use excludeName parameter to avoid adding bridges we don't care about, eg. 
virbr0 */
-char *getBridge(char *excludeName)
+void getBridge(char *excludeName, char *result, size_t resultLen)
 {
        struct dirent *de;
        DIR *d;
 
-       char tmp[256] = { 0 }, *bridge;
-
-       bridge = (char *)malloc(16 * sizeof(char));
+       char tmp[256] = { 0 };
 
        d = opendir("/sys/class/net");
        while ((de = readdir(d)) != NULL) {
@@ -79,14 +77,14 @@ char *getBridge(char *excludeName)
                        && (strstr(de->d_name, excludeName) == NULL)) {
                                sprintf(tmp, "/sys/class/net/%s/bridge", 
de->d_name);
 
-                               if (access(tmp, F_OK) == 0)
-                                       bridge = de->d_name;
+                               if (access(tmp, F_OK) == 0) {
+                                       strncpy(result, de->d_name, resultLen - 
1);
+                                       result[resultLen - 1] = 0;
+                               }
                }
        }
 
        closedir(d);
-
-       return bridge;
 }
 
 /* parseNetLine provides regular expression based parsing for lines from 
/proc/net/dev, all the */
@@ -279,7 +277,7 @@ int xenstat_collect_networks(xenstat_node * node)
              SEEK_SET);
 
        /* We get the bridge devices for use with bonding interface to get 
bonding interface stats */
-       snprintf(devBridge, 16, "%s", getBridge("vir"));
+       getBridge("vir", devBridge, sizeof(devBridge));
        snprintf(devNoBridge, 16, "p%s", devBridge);
 
        while (fgets(line, 512, priv->procnetdev)) {
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.