1
0
mirror of https://git.FreeBSD.org/ports.git synced 2024-12-23 04:23:08 +00:00
freebsd-ports/devel/glib20/files/patch-glib_gmain.c
Koop Mast 86ad7366f2 Add patch from upstream to fix crasher in open-vm-tools [1]
Add back depend on libiconv. This is needed because our base iconv
doesn't do utf-8 -> wchar_t apparently. [2] wchar_t is used by glibmm.

While here:
Use new lib_depend syntax, strip libraries, stagify [3]

Submitted by:	swills@ [1]
		mandree@ [2]
Discussion at:	http://lists.freebsd.org/pipermail/freebsd-ports/2014-February/089603.html [2]
Obtained from:	gnome devel-repo [3]
2014-02-16 15:40:34 +00:00

39 lines
1.3 KiB
C

From 0513c855cba69b96e4c9d2200719249a8a0c586e Mon Sep 17 00:00:00 2001
From: Dan Winship <danw@gnome.org>
Date: Wed, 10 Apr 2013 11:39:12 -0400
Subject: [PATCH] gmain: fix double-unlock in g_main_context_unref()
When unreffing a context with sources still attached, it would end up
unlocking an already-unlocked context, causing crashes on platforms
that (unlike Linux) actually check for that.
https://bugzilla.gnome.org/show_bug.cgi?id=697595
---
glib/gmain.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/glib/gmain.c b/glib/gmain.c
index 68a7f8e..d6037ba 100644
--- glib/gmain.c
+++ glib/gmain.c
@@ -520,12 +520,16 @@ g_main_context_unref (GMainContext *context)
main_context_list = g_slist_remove (main_context_list, context);
G_UNLOCK (main_context_list);
+ /* g_source_iter_next() assumes the context is locked. */
+ LOCK_CONTEXT (context);
g_source_iter_init (&iter, context, TRUE);
while (g_source_iter_next (&iter, &source))
{
source->context = NULL;
- g_source_destroy_internal (source, context, FALSE);
+ g_source_destroy_internal (source, context, TRUE);
}
+ UNLOCK_CONTEXT (context);
+
for (sl_iter = context->source_lists; sl_iter; sl_iter = sl_iter->next)
{
list = sl_iter->data;
--
1.8.1.4