Missing return value

[vlc] / src / control / core.c

diff --git a/src/control/core.c b/src/control/core.c
index 90ae37c60b4ff779cd2f56e69b83419e8ad6cd36..a96ff039d0688055a0d2e8b929836a74fa7246fa 100644 (file)
--- a/src/control/core.c
+++ b/src/control/core.c
@@ -25,6 +25,7 @@
 #include <vlc/libvlc.h>
 
 #include <vlc_interface.h>
+#include <vlc_vlm.h>
 
 #include <stdarg.h>
 #include <limits.h>
@@ -70,6 +71,7 @@ static void libvlc_exception_not_handled( const char *psz )
 {
     fprintf( stderr, "*** LibVLC Exception not handled: %s\nSet a breakpoint in '%s' to debug.\n",
              psz, __func__ );
+    abort();
 }
 
 void libvlc_exception_raise( libvlc_exception_t *p_exception,
@@ -89,6 +91,8 @@ void libvlc_exception_raise( libvlc_exception_t *p_exception,
         /* Print something, so that lazy third-parties can easily
          * notice that something may have gone unnoticedly wrong */
         libvlc_exception_not_handled( psz );
+        if( psz != nomemstr )
+            free( psz );
         return;
     }
 
@@ -135,7 +139,9 @@ libvlc_instance_t * libvlc_new( int argc, const char *const *argv,
     }
 
     p_new->p_libvlc_int = p_libvlc_int;
-    p_new->p_vlm = NULL;
+    p_new->libvlc_vlm.p_vlm = NULL;
+    p_new->libvlc_vlm.p_event_manager = NULL;
+    p_new->libvlc_vlm.pf_release = NULL;
     p_new->b_playlist_locked = 0;
     p_new->ref_count = 1;
     p_new->verbosity = 1;
@@ -161,9 +167,8 @@ void libvlc_release( libvlc_instance_t *p_instance )
     vlc_mutex_t *lock = &p_instance->instance_lock;
     int refs;
 
-    assert( p_instance->ref_count > 0 );
-
     vlc_mutex_lock( lock );
+    assert( p_instance->ref_count > 0 );
     refs = --p_instance->ref_count;
     vlc_mutex_unlock( lock );
 
@@ -171,17 +176,23 @@ void libvlc_release( libvlc_instance_t *p_instance )
     {
         vlc_mutex_destroy( lock );
         vlc_mutex_destroy( &p_instance->event_callback_lock );
+        if( p_instance->libvlc_vlm.pf_release )
+            p_instance->libvlc_vlm.pf_release( p_instance );
         libvlc_InternalCleanup( p_instance->p_libvlc_int );
         libvlc_InternalDestroy( p_instance->p_libvlc_int );
         free( p_instance );
     }
 }
 
-void libvlc_add_intf( libvlc_instance_t *p_i, const char *name,
+int libvlc_add_intf( libvlc_instance_t *p_i, const char *name,
                       libvlc_exception_t *p_e )
 {
     if( libvlc_InternalAddIntf( p_i->p_libvlc_int, name ) )
-        RAISEVOID( "Interface initialization failed" );
+    {
+        libvlc_exception_raise( p_e, "Interface initialization failed" );
+        return -1;
+    }
+    return 0;
 }
 
 void libvlc_wait( libvlc_instance_t *p_i )
@@ -190,12 +201,6 @@ void libvlc_wait( libvlc_instance_t *p_i )
     libvlc_InternalWait( p_libvlc );
 }
 
-int libvlc_get_vlc_id( libvlc_instance_t *p_instance )
-{
-    assert( p_instance );
-    return 1;
-}
-
 const char * libvlc_get_version(void)
 {
     return VLC_Version();
@@ -208,7 +213,8 @@ const char * libvlc_get_compiler(void)
 
 const char * libvlc_get_changeset(void)
 {
-    return "exported";
+    extern const char psz_vlc_changeset[];
+    return psz_vlc_changeset;
 }
 
 /* export internal libvlc_instance for ugly hacks with libvlccore */