From: Markus Armbruster <armbru@redhat.com>
Date: Wed, 28 May 2014 09:17:05 +0000 (+0200)
Subject: block/vvfat: Plug memory leak in read_directory()
X-Git-Tag: v1.7.2~28
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d3cd48a85fc8f0aa4358a866da31842480edf1e6;p=thirdparty%2Fqemu.git

block/vvfat: Plug memory leak in read_directory()

Has always been leaky.  Spotted by Coverity.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
(cherry picked from commit b122c3b6d020e529b203836efb8f611ece787293)
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
---

diff --git a/block/vvfat.c b/block/vvfat.c
index e71d71ead8a..e9e4fad44dd 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -788,7 +788,9 @@ static int read_directory(BDRVVVFATState* s, int mapping_index)
 	    s->current_mapping->path=buffer;
 	    s->current_mapping->read_only =
 		(st.st_mode & (S_IWUSR | S_IWGRP | S_IWOTH)) == 0;
-	}
+        } else {
+            g_free(buffer);
+        }
     }
     closedir(dir);