automount: fix automount with active screensaver and memleak (#426)

* automount: fix queue iteration

fix use-after-free bug caused by dereferencing l after g_list_remove.

This change also fixes auto-mounting multiple volumes that were
connected while the screensaver was active.

* automount: fix memleak in csd-autorun

guessed_content_type also has to be freed if non-NULL (no error) but empty.

Author: okaestne

plugins/automount/csd-automount-manager.c

@@ -190,10 +190,8 @@ check_volume_queue (CsdAutomountManager *manager)
                         g_list_remove (manager->priv->volume_queue, volume);
 
                 g_object_unref (volume);
-                l = l->next;
+                l = manager->priv->volume_queue;
         }
-
-        manager->priv->volume_queue = NULL;
 }
 
 static void

plugins/automount/csd-autorun.c

@@ -798,7 +798,7 @@ autorun_guessed_content_type_callback (GObject *source_object,
 				       gpointer user_data)
 {
 	GError *error;
-	char **guessed_content_type;
+	g_auto(GStrv) guessed_content_type = NULL;
 	AutorunData *data = user_data;
 	gboolean open_folder;
 
@@ -822,7 +822,6 @@ autorun_guessed_content_type_callback (GObject *source_object,
 					open_folder = TRUE;
 				}
 			}
-			g_strfreev (guessed_content_type);
 		} else {
 			if (g_settings_get_boolean (data->settings, "automount-open")) {
 				open_folder = TRUE;