Clean up mode tracking to avoid a small memory leak.

Signed-off-by: Philippe De Swert <philippe.deswert@jollamobile.com>

src/usb_moded-dbus.c

@@ -74,10 +74,11 @@ static DBusHandlerResult msg_handler(DBusConnection *const connection, DBusMessa
 
     	if(!strcmp(member, USB_MODE_STATE_REQUEST))
     	{
-      		const char *mode = get_usb_mode();
+		const char *mode = strdup(get_usb_mode());
 
       		if((reply = dbus_message_new_method_return(msg)))
         		dbus_message_append_args (reply, DBUS_TYPE_STRING, &mode, DBUS_TYPE_INVALID);
+		free((void *)mode);
     	}
     	else if(!strcmp(member, USB_MODE_STATE_SET))
     	{

src/usb_moded.c

@@ -322,8 +322,7 @@ void set_usb_mode(const char *mode)
   }
   if(net)
     log_debug("Network setting failed!\n");
-  free(current_mode.mode);
-  current_mode.mode = strdup(mode);
+  current_mode.mode = mode;
   usb_moded_send_signal(get_usb_mode());
 }
 
@@ -406,8 +405,8 @@ inline const char * get_usb_mode(void)
  */
 void set_usb_module(const char *module)
 {
-  free(current_mode.module);
-  current_mode.module = strdup(module);
+  //free(current_mode.module);
+  current_mode.module = module;
 }
 
 /** get the supposedly loaded module
@@ -469,8 +468,8 @@ static void usb_moded_init(void)
 
   current_mode.connected = FALSE;
   current_mode.mounted = FALSE;
-  current_mode.mode = strdup(MODE_UNDEFINED);
-  current_mode.module = strdup(MODULE_NONE);
+  current_mode.mode = MODE_UNDEFINED;
+  current_mode.module = MODULE_NONE;
 
   /* check config, merge or create if outdated */
   if(conf_file_merge() != 0)
@@ -518,8 +517,7 @@ static gboolean charging_fallback(gpointer data)
   /* since this is the fallback, we keep an indication
      for the UI, as we are not really in charging mode.
   */
-  free(current_mode.mode);
-  current_mode.mode = strdup(MODE_ASK);
+  current_mode.mode = MODE_ASK;
   current_mode.data = NULL;
   charging_timeout = 0;
   log_info("Falling back on charging mode.\n");

src/usb_moded.h

@@ -48,8 +48,8 @@ typedef struct usb_mode
   /*@{*/
   gboolean connected; 		/* connection status, 1 for connected */
   gboolean mounted;  		/* mount status, 1 for mounted -UNUSED atm- */
-  char *mode;  			/* the mode name */
-  char *module; 		/* the module name for the specific mode */
+  const char *mode;		/* the mode name */
+  const char *module; 		/* the module name for the specific mode */
   struct mode_list_elem *data;  /* contains the mode data */
   /*@}*/
 }usb_mode;