+ { + /* check that there really is such IndicatorObjectEntry */ + if (g_hash_table_lookup (priv->entry2indicator_hash, entry) != NULL) + { + return entry; + }
Can you add an extra check in there:
+ if (!INDICATOR_IS_OBJECT_ENTRY (entry)) + { + g_critical ("Invalid IndicatorObjectEntry reference %p", entry); + return NULL; + }
Not that it makes the code much more safe, but it has a slim chance of making debugging easier should we ever see it again
« Back to merge proposal
+ { Entry */ table_lookup (priv-> entry2indicator _hash, entry) != NULL)
+ /* check that there really is such IndicatorObject
+ if (g_hash_
+ {
+ return entry;
+ }
Can you add an extra check in there:
+ if (!INDICATOR_ IS_OBJECT_ ENTRY (entry)) Entry reference %p", entry);
+ {
+ g_critical ("Invalid IndicatorObject
+ return NULL;
+ }
Not that it makes the code much more safe, but it has a slim chance of
making debugging easier should we ever see it again