[tracker] libtracker-miner: Don't crash if TrackerMonitor can't create a sample monitor

[Carlos Garnacho <carlosg src gnome org>]

commit 7831184f0a6d1bbc0ab3ca3365ef05b3dd107ab8
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri May 13 17:05:25 2011 +0200

    libtracker-miner: Don't crash if TrackerMonitor can't create a sample monitor
    
    Neither G_OBJECT_TYPE() or g_type_name() can handle NULL, leading to a crash,
    now a critical warning is issued instead.

 src/libtracker-miner/tracker-monitor.c |   30 +++++++++++++++++++-----------
 1 files changed, 19 insertions(+), 11 deletions(-)
---
diff --git a/src/libtracker-miner/tracker-monitor.c b/src/libtracker-miner/tracker-monitor.c
index b56ae46..737b436 100644
--- a/src/libtracker-miner/tracker-monitor.c
+++ b/src/libtracker-miner/tracker-monitor.c
@@ -220,6 +220,7 @@ tracker_monitor_init (TrackerMonitor *object)
 	GFile                 *file;
 	GFileMonitor          *monitor;
 	const gchar           *name;
+	GError                *error = NULL;
 
 	object->private = TRACKER_MONITOR_GET_PRIVATE (object);
 
@@ -253,16 +254,23 @@ tracker_monitor_init (TrackerMonitor *object)
 	monitor = g_file_monitor_directory (file,
 	                                    G_FILE_MONITOR_WATCH_MOUNTS,
 	                                    NULL,
-	                                    NULL);
+	                                    &error);
 
-	priv->monitor_backend = G_OBJECT_TYPE (monitor);
+	if (error) {
+		g_critical ("Could not create sample directory monitor: %s", error->message);
+		g_error_free (error);
+
+		/* Guessing limit... */
+		priv->monitor_limit = 100;
+	} else {
+		priv->monitor_backend = G_OBJECT_TYPE (monitor);
+
+		/* We use the name because the type itself is actually
+		 * private and not available publically. Note this is
+		 * subject to change, but unlikely of course.
+		 */
+		name = g_type_name (priv->monitor_backend);
 
-	/* We use the name because the type itself is actually
-	 * private and not available publically. Note this is
-	 * subject to change, but unlikely of course.
-	 */
-	name = g_type_name (priv->monitor_backend);
-	if (name) {
 		/* Set limits based on backend... */
 		if (strcmp (name, "GInotifyDirectoryMonitor") == 0) {
 			/* Using inotify */
@@ -319,12 +327,12 @@ tracker_monitor_init (TrackerMonitor *object)
 			/* Guessing limit... */
 			priv->monitor_limit = 100;
 		}
+
+		g_file_monitor_cancel (monitor);
+		g_object_unref (monitor);
 	}
 
-	g_file_monitor_cancel (monitor);
-	g_object_unref (monitor);
 	g_object_unref (file);
-
 	g_message ("Monitor limit is %d", priv->monitor_limit);
 }