Commit b1d6e1fc authored by Rémi Duraffort's avatar Rémi Duraffort Committed by Jean-Baptiste Kempf

pulse: fix a memory leak.

(cherry picked from commit 77dd16bace3e70599cab1a36fe94565bca717227)
Signed-off-by: default avatarJean-Baptiste Kempf <jb@videolan.org>
parent 29a1f252
...@@ -119,7 +119,7 @@ static int Open ( vlc_object_t *p_this ) ...@@ -119,7 +119,7 @@ static int Open ( vlc_object_t *p_this )
const struct pa_buffer_attr *buffer_attr; const struct pa_buffer_attr *buffer_attr;
struct pa_buffer_attr a; struct pa_buffer_attr a;
struct pa_channel_map map; struct pa_channel_map map;
char * p_client_name = NULL; char * p_client_name;
if( !vlc_xlib_init( p_this ) ) if( !vlc_xlib_init( p_this ) )
return VLC_EGENERIC; return VLC_EGENERIC;
...@@ -214,7 +214,10 @@ static int Open ( vlc_object_t *p_this ) ...@@ -214,7 +214,10 @@ static int Open ( vlc_object_t *p_this )
goto fail; goto fail;
} }
if (!(p_sys->context = pa_context_new(pa_threaded_mainloop_get_api(p_sys->mainloop), p_client_name))) { p_sys->context = pa_context_new(pa_threaded_mainloop_get_api(p_sys->mainloop), p_client_name);
free(p_client_name);
if(!p_sys->context)
{
msg_Err(p_aout, "Failed to allocate context"); msg_Err(p_aout, "Failed to allocate context");
goto fail; goto fail;
} }
...@@ -307,7 +310,6 @@ unlock_and_fail: ...@@ -307,7 +310,6 @@ unlock_and_fail:
if (p_sys->mainloop) if (p_sys->mainloop)
pa_threaded_mainloop_unlock(p_sys->mainloop); pa_threaded_mainloop_unlock(p_sys->mainloop);
fail: fail:
free(p_client_name);
msg_Dbg(p_aout, "Pulse initialization failed"); msg_Dbg(p_aout, "Pulse initialization failed");
uninit(p_aout); uninit(p_aout);
return VLC_EGENERIC; return VLC_EGENERIC;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment