Don't leak every https parameters.

[vlc] / modules / control / lirc.c

diff --git a/modules/control/lirc.c b/modules/control/lirc.c
index c11ace77a73e18567bbb9bd90d5a10f790287e58..0ae6fbb1faf8351274d0a73a39aa11e37fac18ed 100644 (file)
--- a/modules/control/lirc.c
+++ b/modules/control/lirc.c
@@ -24,12 +24,15 @@
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <stdlib.h>                                      /* malloc(), free() */
-#include <string.h>
 
 #include <fcntl.h>
 
-#include <vlc/vlc.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
+#include <vlc_plugin.h>
 #include <vlc_interface.h>
 #include <vlc_osd.h>
 
@@ -60,15 +63,15 @@ static void Run     ( intf_thread_t * );
  * Module descriptor
  *****************************************************************************/
 vlc_module_begin();
-    set_shortname( _("Infrared") );
+    set_shortname( N_("Infrared") );
     set_category( CAT_INTERFACE );
     set_subcategory( SUBCAT_INTERFACE_CONTROL );
-    set_description( _("Infrared remote control interface") );
+    set_description( N_("Infrared remote control interface") );
     set_capability( "interface", 0 );
     set_callbacks( Open, Close );
 
     add_string( "lirc-file", NULL, NULL,
-                LIRC_TEXT, LIRC_LONGTEXT, VLC_TRUE );
+                LIRC_TEXT, LIRC_LONGTEXT, true );
 vlc_module_end();
 
 /*****************************************************************************
@@ -89,12 +92,13 @@ static int Open( vlc_object_t *p_this )
 
     p_intf->pf_run = Run;
 
-    p_intf->p_sys->psz_file = var_CreateGetString( p_intf, "lirc-file" );
+    p_intf->p_sys->psz_file = var_CreateGetNonEmptyString( p_intf, "lirc-file" );
 
     i_fd = lirc_init( "vlc", 1 );
     if( i_fd == -1 )
     {
         msg_Err( p_intf, "lirc initialisation failed" );
+        free( p_intf->p_sys->psz_file );
         free( p_intf->p_sys );
         return 1;
     }
@@ -106,6 +110,7 @@ static int Open( vlc_object_t *p_this )
     {
         msg_Err( p_intf, "failure while reading lirc config" );
         lirc_deinit();
+        free( p_intf->p_sys->psz_file );
         free( p_intf->p_sys );
         return 1;
     }
@@ -121,8 +126,7 @@ static void Close( vlc_object_t *p_this )
     intf_thread_t *p_intf = (intf_thread_t *)p_this;
 
     /* Destroy structure */
-    if( p_intf->p_sys->psz_file )
-        free( p_intf->p_sys->psz_file );
+    free( p_intf->p_sys->psz_file );
     lirc_freeconfig( p_intf->p_sys->config );
     lirc_deinit();
     free( p_intf->p_sys );