mirror of
https://github.com/gentoo-mirror/gentoo.git
synced 2026-01-04 01:37:34 -08:00
56bd759df1
This commit represents a new era for Gentoo: Storing the gentoo-x86 tree in Git, as converted from CVS. This commit is the start of the NEW history. Any historical data is intended to be grafted onto this point. Creation process: 1. Take final CVS checkout snapshot 2. Remove ALL ChangeLog* files 3. Transform all Manifests to thin 4. Remove empty Manifests 5. Convert all stale $Header$/$Id$ CVS keywords to non-expanded Git $Id$ 5.1. Do not touch files with -kb/-ko keyword flags. Signed-off-by: Robin H. Johnson <robbat2@gentoo.org> X-Thanks: Alec Warner <antarus@gentoo.org> - did the GSoC 2006 migration tests X-Thanks: Robin H. Johnson <robbat2@gentoo.org> - infra guy, herding this project X-Thanks: Nguyen Thai Ngoc Duy <pclouds@gentoo.org> - Former Gentoo developer, wrote Git features for the migration X-Thanks: Brian Harring <ferringb@gentoo.org> - wrote much python to improve cvs2svn X-Thanks: Rich Freeman <rich0@gentoo.org> - validation scripts X-Thanks: Patrick Lauer <patrick@gentoo.org> - Gentoo dev, running new 2014 work in migration X-Thanks: Michał Górny <mgorny@gentoo.org> - scripts, QA, nagging X-Thanks: All of other Gentoo developers - many ideas and lots of paint on the bikeshed
66 lines
2.0 KiB
Diff
66 lines
2.0 KiB
Diff
From cc14440eface093548cb3bc7814da11d9a99d283 Mon Sep 17 00:00:00 2001
|
|
From: Anssi Hannula <anssi@mageia.org>
|
|
Date: Wed, 4 Jan 2012 00:23:55 +0200
|
|
Subject: [PATCH] fix possible server deadlock in ih_sub_cancel
|
|
|
|
ih_sub_foreach() calls ih_sub_cancel() while inotify_lock is locked.
|
|
However, ih_sub_cancel() locks it again, and locking GMutex recursively
|
|
causes undefined behaviour.
|
|
|
|
Fix that by removing locking from ih_sub_cancel() as ih_sub_foreach()
|
|
is its only user. Also make the function static so that it won't
|
|
accidentally get used by other files without locking (inotify-helper.h
|
|
is an internal server header).
|
|
|
|
This should fix the intermittent deadlocks I've been experiencing
|
|
causing KDE applications to no longer start, and probably also
|
|
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=542361
|
|
---
|
|
server/inotify-helper.c | 7 ++-----
|
|
server/inotify-helper.h | 1 -
|
|
2 files changed, 2 insertions(+), 6 deletions(-)
|
|
|
|
diff --git a/server/inotify-helper.c b/server/inotify-helper.c
|
|
index d77203e..0789fa4 100644
|
|
--- a/server/inotify-helper.c
|
|
+++ b/server/inotify-helper.c
|
|
@@ -123,13 +123,11 @@ ih_sub_add (ih_sub_t * sub)
|
|
|
|
/**
|
|
* Cancels a subscription which was being monitored.
|
|
+ * inotify_lock must be held when calling.
|
|
*/
|
|
-gboolean
|
|
+static gboolean
|
|
ih_sub_cancel (ih_sub_t * sub)
|
|
{
|
|
- G_LOCK(inotify_lock);
|
|
-
|
|
-
|
|
if (!sub->cancelled)
|
|
{
|
|
IH_W("cancelling %s\n", sub->pathname);
|
|
@@ -140,7 +138,6 @@ ih_sub_cancel (ih_sub_t * sub)
|
|
sub_list = g_list_remove (sub_list, sub);
|
|
}
|
|
|
|
- G_UNLOCK(inotify_lock);
|
|
return TRUE;
|
|
}
|
|
|
|
diff --git a/server/inotify-helper.h b/server/inotify-helper.h
|
|
index 5d3b6d0..d36b5fd 100644
|
|
--- a/server/inotify-helper.h
|
|
+++ b/server/inotify-helper.h
|
|
@@ -34,7 +34,6 @@ gboolean ih_startup (event_callback_t ecb,
|
|
found_callback_t fcb);
|
|
gboolean ih_running (void);
|
|
gboolean ih_sub_add (ih_sub_t *sub);
|
|
-gboolean ih_sub_cancel (ih_sub_t *sub);
|
|
|
|
/* Return FALSE from 'f' if the subscription should be cancelled */
|
|
void ih_sub_foreach (void *callerdata, gboolean (*f)(ih_sub_t *sub, void *callerdata));
|
|
--
|
|
1.7.7.2
|
|
|