From 29c145bcbe26d2806ae1ba74dae8f95d39611ff6 Mon Sep 17 00:00:00 2001 From: John Scipione Date: Fri, 18 Jul 2014 14:10:38 -0400 Subject: [PATCH] Tracker: Check LockLooper() return value. Fixes CID602468 and CID602469 Thanks waddlesplash for pointing this out. --- src/kits/tracker/DesktopPoseView.cpp | 22 ++++++++++++---------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/src/kits/tracker/DesktopPoseView.cpp b/src/kits/tracker/DesktopPoseView.cpp index beaba0882c..2e259c3f20 100644 --- a/src/kits/tracker/DesktopPoseView.cpp +++ b/src/kits/tracker/DesktopPoseView.cpp @@ -199,22 +199,24 @@ DesktopPoseView::ShowVolumes(bool visible, bool showShared) void DesktopPoseView::StartSettingsWatch() { - be_app->LockLooper(); - be_app->StartWatching(this, kShowDisksIconChanged); - be_app->StartWatching(this, kVolumesOnDesktopChanged); - be_app->StartWatching(this, kDesktopIntegrationChanged); - be_app->UnlockLooper(); + if (be_app->LockLooper()) { + be_app->StartWatching(this, kShowDisksIconChanged); + be_app->StartWatching(this, kVolumesOnDesktopChanged); + be_app->StartWatching(this, kDesktopIntegrationChanged); + be_app->UnlockLooper(); + } } void DesktopPoseView::StopSettingsWatch() { - be_app->LockLooper(); - be_app->StopWatching(this, kShowDisksIconChanged); - be_app->StopWatching(this, kVolumesOnDesktopChanged); - be_app->StopWatching(this, kDesktopIntegrationChanged); - be_app->UnlockLooper(); + if (be_app->LockLooper()) { + be_app->StopWatching(this, kShowDisksIconChanged); + be_app->StopWatching(this, kVolumesOnDesktopChanged); + be_app->StopWatching(this, kDesktopIntegrationChanged); + be_app->UnlockLooper(); + } }