From 4f8c8d97bf2fd35eb16fa7dd6c8c587c2fc19f96 Mon Sep 17 00:00:00 2001
From: Stuart Morgan <smorgan@mythtv.org>
Date: Sun, 9 Jun 2013 16:33:47 +0100
Subject: [PATCH] Fix coverity 1026714. Modulo by zero. In fact an operator
 precedence error, fixed by applying brackets in the correct place.

---
 mythtv/libs/libmythfreemheg/Programs.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mythtv/libs/libmythfreemheg/Programs.cpp b/mythtv/libs/libmythfreemheg/Programs.cpp
index f77385d6545..60d78164e46 100644
--- a/mythtv/libs/libmythfreemheg/Programs.cpp
+++ b/mythtv/libs/libmythfreemheg/Programs.cpp
@@ -384,7 +384,7 @@ void MHResidentProgram::CallProgram(bool fIsFork, const MHObjectRef &success, co
             {
                 int nLimit = GetInt(args.GetAt(0), engine);
                 MHParameter *pResInt = args.GetAt(1);
-                int r = random() % nLimit + 1;
+                int r = random() % (nLimit + 1);
                 engine->FindObject(
                     *(pResInt->GetReference()))->SetVariableValue(r);
                 SetSuccessFlag(success, true, engine);