summaryrefslogtreecommitdiffstats
path: root/src/server/events/Delete.cpp
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2011-09-24 03:22:30 +0000
committerDavid Robillard <d@drobilla.net>2011-09-24 03:22:30 +0000
commit74a711e0d1cdb5c505a227dc9b1925657f1e778d (patch)
tree78f93b26e4154c76892cb0160414c4effbfc730e /src/server/events/Delete.cpp
parent2be10b0b6f2c0f01870208e9d18e5db87e5dfb88 (diff)
downloadingen-74a711e0d1cdb5c505a227dc9b1925657f1e778d.tar.gz
ingen-74a711e0d1cdb5c505a227dc9b1925657f1e778d.tar.bz2
ingen-74a711e0d1cdb5c505a227dc9b1925657f1e778d.zip
Use store lock to avoid race conditions with Get and create/delete events.
Get really shouldn't be reading the store (via ObjectSender) in the post processing thread at all, avoiding that entirely would be a better solution. git-svn-id: http://svn.drobilla.net/lad/trunk/ingen@3484 a436a847-0d15-0410-975c-d299462d15a1
Diffstat (limited to 'src/server/events/Delete.cpp')
-rw-r--r--src/server/events/Delete.cpp4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/server/events/Delete.cpp b/src/server/events/Delete.cpp
index b1dc1558..4f13cd97 100644
--- a/src/server/events/Delete.cpp
+++ b/src/server/events/Delete.cpp
@@ -50,6 +50,7 @@ Delete::Delete(Engine& engine,
, _ports_array(NULL)
, _compiled_patch(NULL)
, _disconnect_event(NULL)
+ , _lock(engine.engine_store()->lock(), Glib::NOT_LOCK)
{
assert(request);
assert(request->source());
@@ -71,6 +72,8 @@ Delete::pre_process()
return;
}
+ _lock.acquire();
+
_removed_bindings = _engine.control_bindings()->remove(_path);
_store_iterator = _engine.engine_store()->find(_path);
@@ -170,6 +173,7 @@ Delete::execute(ProcessContext& context)
void
Delete::post_process()
{
+ _lock.release();
_removed_bindings.reset();
if (!Raul::Path::is_path(_uri)