From 676939f61671f96e80323cf929ee506f3844fcfc Mon Sep 17 00:00:00 2001
From: Steffen Sledz <sledz@dresearch.de>
Date: Thu, 5 Aug 2010 07:48:36 +0200
Subject: break potential infinite loop in actionDoRetry

If a module always returns RS_RET_OK (like ompipe does) the actionDoRetry
loop may not have leaved faked ACT_STATE_SUSP state in case iResumeOKinRow
had ever reached a count of 1000.

Signed-off-by: Rainer Gerhards <rgerhards@adiscon.com>
---
 action.c | 1 +
 1 file changed, 1 insertion(+)

(limited to 'action.c')

diff --git a/action.c b/action.c
index 32a07dcb..90ec1bf0 100644
--- a/action.c
+++ b/action.c
@@ -508,6 +508,7 @@ static rsRetVal actionDoRetry(action_t *pThis, time_t ttNow)
 		iRet = pThis->pMod->tryResume(pThis->pModData);
 		if((pThis->iResumeOKinRow > 999) && (pThis->iResumeOKinRow % 1000 == 0)) {
 			bTreatOKasSusp = 1;
+			pThis->iResumeOKinRow = 0;
 		} else {
 			bTreatOKasSusp = 0;
 		}
-- 
cgit