commit 2eb6c6fd6f21bb3a9433ad2e63a1d2398102da19
author P Dheeraj Srujan Kumar <p.dheeraj.srujan.kumar@intel.com> Mon Jan 23 20:57:16 2023 +0530
committer P Dheeraj Srujan Kumar <p.dheeraj.srujan.kumar@intel.com> Mon Jan 23 20:57:16 2023 +0530
tree 3f4163106dfdcbd8331f2e9d3355cef3a6e6a35a
parent 9e92c96b323160cb81d9964e3aeda5f2687fda63

Error handling for modulo operation

Add zero check for denominator as it could result in divide by zero
exception.

Tested:
Successfully able to handle the zero condition

Change-Id: I400b03db1fbf81afdaa92243fa4af48d19807466
Signed-off-by: P Dheeraj Srujan Kumar <p.dheeraj.srujan.kumar@intel.com>

src/expression.cpp

diff --git a/src/expression.cpp b/src/expression.cpp
index c859800..377bdd8 100644
--- a/src/expression.cpp
+++ b/src/expression.cpp
@@ -75,6 +75,11 @@
         }
         case Operation::modulo:
         {
+            if (b == 0)
+            {
+                throw std::runtime_error(
+                    "Math error: Attempted to divide by Zero\n");
+            }
             return a % b;
         }