From bf1cae28261036ea41ec58462ff747ed0218f4d3 Mon Sep 17 00:00:00 2001
From: Rob Swindell <rob@synchro.net>
Date: Sat, 30 Apr 2022 14:13:46 -0700
Subject: [PATCH] Fix CID 174496: Integer handling issues (BAD_SHIFT)
Don't repeat call atoi() unnecessarily.
---
src/sbbs3/scfg/scfgxtrn.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/sbbs3/scfg/scfgxtrn.c b/src/sbbs3/scfg/scfgxtrn.c
index 320b1cf69f..adece05dcf 100644
--- a/src/sbbs3/scfg/scfgxtrn.c
+++ b/src/sbbs3/scfg/scfgxtrn.c
@@ -666,8 +666,9 @@ void tevents_cfg()
,str,50,K_EDIT);
cfg.event[i]->months=0;
for(p=str;*p;p++) {
- if(atoi(p)) {
- cfg.event[i]->months|=(1<<(atoi(p)-1));
+ int num = atoi(p);
+ if(num > 0) {
+ cfg.event[i]->months|=(1<<(num-1));
while(*p && IS_DIGIT(*p))
p++;
} else {
--
GitLab