Commit e82d21d7 authored by Rémi Duraffort's avatar Rémi Duraffort

factorisation and add a missing dbus_connection_unref.

parent e5f21614
...@@ -103,7 +103,7 @@ static int Open( vlc_object_t *p_this ) ...@@ -103,7 +103,7 @@ static int Open( vlc_object_t *p_this )
return VLC_ENOMEM; return VLC_ENOMEM;
DBusError dbus_error; DBusError dbus_error;
DBusConnection *p_connection; DBusConnection *p_connection = NULL;
p_sd_global = p_sd; p_sd_global = p_sd;
p_sys->i_devices_number = 0; p_sys->i_devices_number = 0;
...@@ -124,10 +124,7 @@ static int Open( vlc_object_t *p_this ) ...@@ -124,10 +124,7 @@ static int Open( vlc_object_t *p_this )
if( dbus_error_is_set( &dbus_error ) ) if( dbus_error_is_set( &dbus_error ) )
{ {
msg_Err( p_sd, "unable to connect to DBUS: %s", dbus_error.message ); msg_Err( p_sd, "unable to connect to DBUS: %s", dbus_error.message );
dbus_error_free( &dbus_error ); goto error;
libhal_ctx_free( p_sys->p_ctx );
free( p_sys );
return VLC_EGENERIC;
} }
libhal_ctx_set_dbus_connection( p_sys->p_ctx, p_connection ); libhal_ctx_set_dbus_connection( p_sys->p_ctx, p_connection );
p_sys->p_connection = p_connection; p_sys->p_connection = p_connection;
...@@ -149,6 +146,8 @@ static int Open( vlc_object_t *p_this ) ...@@ -149,6 +146,8 @@ static int Open( vlc_object_t *p_this )
return VLC_SUCCESS; return VLC_SUCCESS;
error: error:
if( p_connection )
dbus_connection_unref( p_connection );
dbus_error_free( &dbus_error ); dbus_error_free( &dbus_error );
libhal_ctx_free( p_sys->p_ctx ); libhal_ctx_free( p_sys->p_ctx );
free( p_sys ); free( p_sys );
......
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