luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-02-23 18:52:09 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'coverity-signal-emissions' into 'main'

Browse Source 
gsignal: Change signal instance ref semantics under static analysis

See merge request GNOME/glib!2762
This commit is contained in:
Philip Withnall 2022-06-21 12:54:41 +00:00
commit f805635e11
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
gobject/gsignal.c
View File

@ -3402,7 +3402,16 @@ g_signal_emit_valist (gpointer instance,
if (closure != NULL) if (closure != NULL)
{ {
/*
* Coverity doesn’t understand the paired ref/unref here and seems
* to ignore the ref, thus reports every call to g_signal_emit()
* as causing a double-free. That’s incorrect, but I can’t get a
* model file to work for avoiding the false positives, so instead
* comment out the ref/unref when doing static analysis.
*/
#ifndef __COVERITY__
g_object_ref (instance); g_object_ref (instance);
#endif
_g_closure_invoke_va (closure, _g_closure_invoke_va (closure,
return_accu, return_accu,
instance, instance,
@ -3452,8 +3461,11 @@ g_signal_emit_valist (gpointer instance,
TRACE(GOBJECT_SIGNAL_EMIT_END(signal_id, detail, instance, instance_type)); TRACE(GOBJECT_SIGNAL_EMIT_END(signal_id, detail, instance, instance_type));
/* See comment above paired ref above */
#ifndef __COVERITY__
if (closure != NULL) if (closure != NULL)
g_object_unref (instance); g_object_unref (instance);
#endif
return; return;
} }