QA Check Issue: CHECK 31 - 'local' keyword outside function context
Severity: CRITICAL - Causes runtime errors
Problem:
The 'local' keyword can only be used inside bash functions. Using it
at the global scope or inside while loops (but outside functions)
causes "local: can only be used in a function" runtime error.
Found 7 instances:
- Line 1043: flagged_ips (inside heredoc while loop)
- Line 1046: filename (inside heredoc while loop)
- Line 1047: filepath (inside heredoc while loop)
- Line 1060: ip (inside nested while loop #1)
- Line 1078: ip (inside nested while loop #2)
- Line 1171: paths_declaration (outside any function)
- Line 1223: scan_pid (outside any function)
Fix:
Changed all 7 instances from 'local var=' to 'var=' since they are
not inside function scope. These variables are still properly scoped
within their respective while loops or code blocks.
Impact:
- Prevents runtime errors when script executes
- Maintains correct variable scoping
- No functional changes to logic
Verification:
- bash -n syntax check: PASSED
- All 'local' keywords now only appear inside functions
- Script logic unchanged
cschantz
8034ea3b64