From: Matthias Clasen <mclasen@redhat.com>
Date: Sat, 13 May 2006 04:36:45 +0000 (+0000)
Subject: Fix several cases of deref-before-NULL-check. (#341191, Pascal Terjan)
X-Git-Url: http://git.openbox.org/?a=commitdiff_plain;h=f765f65eda41785d9b369067d7492bd1c02ab262;p=dana%2Fcg-glib.git

Fix several cases of deref-before-NULL-check. (#341191, Pascal Terjan)

2006-05-13  Matthias Clasen  <mclasen@redhat.com>

	* glib/grel.c: Fix several cases of deref-before-NULL-check.
	(#341191, Pascal Terjan)
---

diff --git a/ChangeLog b/ChangeLog
index 6c6d5c2a..9d32ce1f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2006-05-13  Matthias Clasen  <mclasen@redhat.com>
 
+	* glib/grel.c: Fix several cases of deref-before-NULL-check.
+	(#341191, Pascal Terjan)
+
 	* glib/glib.symbols: 
 	* glib/goption.h: 
 	* glib/goption.c: Allow optional summary and description
diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12
index 6c6d5c2a..9d32ce1f 100644
--- a/ChangeLog.pre-2-12
+++ b/ChangeLog.pre-2-12
@@ -1,5 +1,8 @@
 2006-05-13  Matthias Clasen  <mclasen@redhat.com>
 
+	* glib/grel.c: Fix several cases of deref-before-NULL-check.
+	(#341191, Pascal Terjan)
+
 	* glib/glib.symbols: 
 	* glib/goption.h: 
 	* glib/goption.c: Allow optional summary and description
diff --git a/glib/grel.c b/glib/grel.c
index 814401b4..b7793be3 100644
--- a/glib/grel.c
+++ b/glib/grel.c
@@ -252,11 +252,15 @@ g_relation_delete  (GRelation     *relation,
 		    gconstpointer  key,
 		    gint           field)
 {
-  GHashTable *table = relation->hashed_tuple_tables[field];
+  GHashTable *table; 
   GHashTable *key_table;
-  gint        count = relation->count;
+  gint        count;
   
   g_return_val_if_fail (relation != NULL, 0);
+
+  table = relation->hashed_tuple_tables[field];
+  count = relation->count;
+
   g_return_val_if_fail (table != NULL, 0);
   
   key_table = g_hash_table_lookup (table, key);
@@ -300,14 +304,18 @@ g_relation_select (GRelation     *relation,
 		   gconstpointer  key,
 		   gint           field)
 {
-  GHashTable  *table = relation->hashed_tuple_tables[field];
+  GHashTable  *table;
   GHashTable  *key_table;
-  GRealTuples *tuples = g_new0 (GRealTuples, 1);
+  GRealTuples *tuples; 
   gint count;
   
   g_return_val_if_fail (relation != NULL, NULL);
+
+  table = relation->hashed_tuple_tables[field];
+
   g_return_val_if_fail (table != NULL, NULL);
   
+  tuples = g_new0 (GRealTuples, 1);
   key_table = g_hash_table_lookup (table, key);
   
   if (!key_table)
@@ -330,10 +338,13 @@ g_relation_count (GRelation     *relation,
 		  gconstpointer  key,
 		  gint           field)
 {
-  GHashTable  *table = relation->hashed_tuple_tables[field];
+  GHashTable  *table;
   GHashTable  *key_table;
   
   g_return_val_if_fail (relation != NULL, 0);
+
+  table = relation->hashed_tuple_tables[field];
+
   g_return_val_if_fail (table != NULL, 0);
   
   key_table = g_hash_table_lookup (table, key);