summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOndrej Holy <oholy@redhat.com>2016-06-09 18:41:58 +0200
committerOndrej Holy <oholy@redhat.com>2016-06-10 14:12:01 +0200
commit9f1584ad558f3dfae7d6490f84c21155fc55e526 (patch)
tree485a4da50f88541e6d69f695d8970ebe9fed8e7a
parent4c2953b0d806062bcce8985a3077e088a48753a5 (diff)
downloadgvfs-9f1584ad558f3dfae7d6490f84c21155fc55e526.tar.gz
afc: Fix several memory leaks
Memory is not released on several places, when handling errors mostly. Add missing g_free calls to avoid those memory leaks.
-rw-r--r--daemon/gvfsbackendafc.c9
1 files changed, 9 insertions, 0 deletions
diff --git a/daemon/gvfsbackendafc.c b/daemon/gvfsbackendafc.c
index 068eb565..2f264ab5 100644
--- a/daemon/gvfsbackendafc.c
+++ b/daemon/gvfsbackendafc.c
@@ -1120,10 +1120,13 @@ not_found_bail:
new_path ? new_path : path, AFC_FOPEN_RDONLY, &fd),
G_VFS_JOB(job))))
{
+ g_free (new_path);
g_free (app);
return;
}
+ g_free (new_path);
+
handle = g_vfs_backend_file_handle_new (self, app);
handle->fd = fd;
handle->afc_cli = afc_cli;
@@ -1168,6 +1171,7 @@ g_vfs_backend_afc_create (GVfsBackend *backend,
info = g_hash_table_lookup (self->apps, app);
if (info == NULL)
{
+ g_free (new_path);
g_free (app);
g_vfs_backend_afc_check (AFC_E_OBJECT_NOT_FOUND, G_VFS_JOB(job));
return;
@@ -1237,6 +1241,7 @@ g_vfs_backend_afc_append_to (GVfsBackend *backend,
info = g_hash_table_lookup (self->apps, app);
if (info == NULL)
{
+ g_free (new_path);
g_free (app);
g_vfs_backend_afc_check (AFC_E_OBJECT_NOT_FOUND, G_VFS_JOB(job));
return;
@@ -1335,6 +1340,7 @@ g_vfs_backend_afc_replace (GVfsBackend *backend,
info = g_hash_table_lookup (self->apps, app);
if (info == NULL)
{
+ g_free (new_path);
g_free (app);
g_vfs_backend_afc_check (AFC_E_OBJECT_NOT_FOUND, G_VFS_JOB(job));
return;
@@ -2617,6 +2623,9 @@ g_vfs_backend_afc_move (GVfsBackend *backend,
info = g_hash_table_lookup (self->apps, app_src);
if (info == NULL)
{
+ g_free (app_src);
+ g_free (new_src);
+ g_free (new_dst);
g_vfs_backend_afc_check (AFC_E_OBJECT_NOT_FOUND, G_VFS_JOB(job));
return;
}