Skip to content

Commit d6f55b1

Browse files
committed
replaced the is_u32 function call with the same numeric validation pattern
Signed-off-by: Nitin Nakka <nitinn@qti.qualcomm.com>
1 parent bf17e92 commit d6f55b1

1 file changed

Lines changed: 15 additions & 4 deletions

File tree

  • Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode

Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/run.sh

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -170,10 +170,21 @@ while [ $# -gt 0 ]; do
170170
echo "$TESTNAME SKIP" >"$RES_FILE"
171171
exit 0
172172
fi
173-
if [ -n "$2" ] && ! is_u32 "$2"; then
174-
log_warn "Invalid --framerate '$2' (must be numeric)"
175-
echo "$TESTNAME SKIP" >"$RES_FILE"
176-
exit 0
173+
if [ -n "$2" ]; then
174+
case "$2" in
175+
''|*[!0-9]*)
176+
log_warn "Invalid --framerate '$2' (must be numeric)"
177+
echo "$TESTNAME SKIP" >"$RES_FILE"
178+
exit 0
179+
;;
180+
*)
181+
if [ "$2" -le 0 ] 2>/dev/null; then
182+
log_warn "Framerate must be positive (got '$2')"
183+
echo "$TESTNAME SKIP" >"$RES_FILE"
184+
exit 0
185+
fi
186+
;;
187+
esac
177188
fi
178189
# If empty, keep default; otherwise use provided value
179190
[ -n "$2" ] && framerate="$2"

0 commit comments

Comments
 (0)