From 1c7c2f50b0c5c2798142c55c32294cf5458d98bd Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Sun, 1 Jan 2017 17:52:51 +0100 Subject: Fixed a possible deadlock in the recordings handler thread --- recording.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) (limited to 'recording.c') diff --git a/recording.c b/recording.c index 5293459e..63980585 100644 --- a/recording.c +++ b/recording.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: recording.c 4.6 2016/12/22 12:58:20 kls Exp $ + * $Id: recording.c 4.7 2017/01/01 17:52:51 kls Exp $ */ #include "recording.h" @@ -1914,16 +1914,20 @@ cRecordingsHandler::~cRecordingsHandler() void cRecordingsHandler::Action(void) { while (Running()) { + bool Sleep = false; { cMutexLock MutexLock(&mutex); - while (cRecordingsHandlerEntry *r = operations.First()) { - if (!r->Active(error)) - operations.Del(r); - } - if (!operations.Count()) + if (cRecordingsHandlerEntry *r = operations.First()) { + if (!r->Active(error)) + operations.Del(r); + else + Sleep = true; + } + else break; } - cCondWait::SleepMs(100); + if (Sleep) + cCondWait::SleepMs(100); } } -- cgit v1.2.3