diff options
| author | Sahil Pai <sahil.pai1@samsung.com> | 2018-05-03 17:35:38 +0530 |
|---|---|---|
| committer | Amit Purwar <amit.purwar@samsung.com> | 2019-04-29 09:35:11 +0530 |
| commit | 3e1a7c201759ca648ca32916fa9f9e963ae13324 (patch) | |
| tree | b7b533910e0c9ecc3957c0bd5dee5c4d4d3bb831 /obexd/src | |
| parent | b2b2d64c81ac0c109233f6356fe9c4b023fb8c37 (diff) | |
| download | bluez-3e1a7c201759ca648ca32916fa9f9e963ae13324.tar.gz bluez-3e1a7c201759ca648ca32916fa9f9e963ae13324.tar.bz2 bluez-3e1a7c201759ca648ca32916fa9f9e963ae13324.zip | |
obexd: Fix memory leak when loading plugins
On failure to open plugin directory, memory allocated to 'patterns'
and 'excludes' is not being freed, and the following memory leak is
logged:
Direct leak of 28 byte(s) in 1 object(s) allocated from:
#0 0xb6aa1c49 in malloc (/usr/lib/liblsan.so+0x9c49)
#1 0xb6976e89 in g_malloc (/lib/libglib-2.0.so.0+0x3ae89)
#2 0xb69886f1 in g_strsplit_set (/lib/libglib-2.0.so.0+0x4c6f1)
#3 0xb6f82f3b in plugin_init (/usr/libexec/bluetooth/obexd+0x20f3b)
#4 0xb6f70dc9 in main (/usr/libexec/bluetooth/obexd+0xedc9)
Change-Id: I9c930ba8fcd4cd04a25f4329ee02c5696c19cf39
Signed-off-by: Amit Purwar <amit.purwar@samsung.com>
Diffstat (limited to 'obexd/src')
| -rwxr-xr-x | obexd/src/plugin.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/obexd/src/plugin.c b/obexd/src/plugin.c index 7d971b6c..a1962b91 100755 --- a/obexd/src/plugin.c +++ b/obexd/src/plugin.c @@ -138,8 +138,11 @@ gboolean plugin_init(const char *pattern, const char *exclude) DBG("Loading plugins %s", PLUGINDIR); dir = g_dir_open(PLUGINDIR, 0, NULL); - if (!dir) + if (!dir) { + g_strfreev(patterns); + g_strfreev(excludes); return FALSE; + } while ((file = g_dir_read_name(dir)) != NULL) { struct obex_plugin_desc *desc; |