diff options
author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2011-07-21 13:06:21 +0100 |
---|---|---|
committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2011-09-21 11:33:38 +0100 |
commit | 9e93d82862c4c9af9f2da9cba74b70ff98fd3b60 (patch) | |
tree | afb3ba39b6ea1dc91c270356cafbc139c336f440 /bus | |
parent | 4dd6abc9535dcf920cf2b0757769e6450c235d49 (diff) | |
download | dbus-9e93d82862c4c9af9f2da9cba74b70ff98fd3b60.tar.gz |
update_desktop_file_entry: don't leak file_path on one particular OOM
Revenge of #33126: most, but not all, temporary variables were freed on
this code path.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=39230
Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
Diffstat (limited to 'bus')
-rw-r--r-- | bus/activation.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/bus/activation.c b/bus/activation.c index 3177d023..c1f0e949 100644 --- a/bus/activation.c +++ b/bus/activation.c @@ -457,8 +457,7 @@ update_desktop_file_entry (BusActivation *activation, * the entries hash table */ _dbus_hash_table_remove_string (entry->s_dir->entries, entry->filename); - bus_activation_entry_unref (entry); - return FALSE; + goto out; } } |