Made psppire-conf thread safer

diff --git a/src/ui/gui/psppire-conf.c b/src/ui/gui/psppire-conf.c
index 4090b74c3ea9dce414e42cb59bb643c6c312a15e..ed2f9178c7c6d26977b0e840bf8632c7e287ae4f 100644 (file)
--- a/src/ui/gui/psppire-conf.c
+++ b/src/ui/gui/psppire-conf.c
@@ -95,6 +95,7 @@ static void
 psppire_conf_finalize (GObject *object)
 {
   PsppireConf *conf = PSPPIRE_CONF (object);
+  g_mutex_free (conf->mutex);
   g_key_file_free (conf->keyfile);
   g_free (conf->filename);
 }
@@ -133,7 +134,6 @@ psppire_conf_class_init (PsppireConfClass *class)
   object_class->finalize = psppire_conf_finalize;
   object_class->dispose = psppire_conf_dispose;
   object_class->constructor = psppire_conf_construct;
-
 }
 
 
@@ -142,6 +142,8 @@ psppire_conf_init (PsppireConf *conf)
 {
   const gchar *dirname = g_get_user_config_dir ();
 
+  conf->mutex = g_mutex_new ();
+
   conf->filename = g_strdup_printf ("%s/%s", dirname, "psppirerc");
 
   conf->keyfile = g_key_file_new ();
@@ -164,6 +166,8 @@ psppire_conf_get_int (PsppireConf *conf, const gchar *base,
 {
   gboolean ok;
   GError *err = NULL;
+
+  g_mutex_lock (conf->mutex);
   conf_read (conf);
   *value = g_key_file_get_integer (conf->keyfile,
                                   base,
@@ -173,6 +177,9 @@ psppire_conf_get_int (PsppireConf *conf, const gchar *base,
   if ( err != NULL )
     g_error_free (err);
 
+
+  g_mutex_unlock (conf->mutex);
+
   return ok;
 }
 
@@ -183,6 +190,8 @@ psppire_conf_get_boolean (PsppireConf *conf, const gchar *base,
   gboolean ok;
   gboolean b;
   GError *err = NULL;
+  g_mutex_lock (conf->mutex);
+
   conf_read (conf);
   b = g_key_file_get_boolean (conf->keyfile,
                              base,
@@ -195,6 +204,8 @@ psppire_conf_get_boolean (PsppireConf *conf, const gchar *base,
   if (ok)
     *value = b;
 
+  g_mutex_unlock (conf->mutex);
+
   return ok;
 }
 
@@ -204,8 +215,10 @@ psppire_conf_set_int (PsppireConf *conf,
                      const gchar *base, const gchar *name,
                      gint value)
 {
+  g_mutex_lock (conf->mutex);
   g_key_file_set_integer (conf->keyfile, base, name, value);
   conf_write (conf);
+  g_mutex_unlock (conf->mutex);
 }
 
 void
@@ -213,8 +226,10 @@ psppire_conf_set_boolean (PsppireConf *conf,
                          const gchar *base, const gchar *name,
                          gboolean value)
 {
+  g_mutex_lock (conf->mutex);
   g_key_file_set_boolean (conf->keyfile, base, name, value);
   conf_write (conf);
+  g_mutex_unlock (conf->mutex);
 }
 
 /*

diff --git a/src/ui/gui/psppire-conf.h b/src/ui/gui/psppire-conf.h
index a7415b2451010b081b0774c9b014c8b205c34613..7e9ab6ad127d1cdf69f04f0ea2daa38b7fc9daaa 100644 (file)
--- a/src/ui/gui/psppire-conf.h
+++ b/src/ui/gui/psppire-conf.h
@@ -61,6 +61,7 @@ struct _PsppireConf
   gboolean dispose_has_run ;
 
   GKeyFile *keyfile;
+  GMutex *mutex;
   gchar *filename;
 };