YETUS-789. bugsystem_linecomments_trigger breaks if prevnum or linenu…

…m isn't numeric Signed-off-by: Sean Busbey <busbey@apache.org>
apache · Feb 2, 2019 · bf88009 · bf88009
1 parent aa16845
commit bf88009
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/precommit/src/main/shell/test-patch.sh b/precommit/src/main/shell/test-patch.sh
@@ -2396,6 +2396,16 @@ function bugsystem_linecomments_trigger
 
     text=$(echo "${line}" | cut -f3- -d:)
 
+    # if, for some reason either one of these
+    # isn't a number, force it to be zero.
+    if [[ ! ${prevnum} =~ ^[0-9]+$ ]]; then
+      prevnum=0
+    fi
+
+    if [[ ! ${linenum} =~ ^[0-9]+$ ]]; then
+      linenum=0
+    fi
+
     if [[ "${prevfn}" == "${fn}" ]] &&
        [[ ${prevnum} -eq ${linenum} ]]; then
       linetext+=("${text}")