st-util cannot parse -V and -F options and --freq option results in a segmentation fault #1428
Comments
sansyse
pushed a commit
to sansyse/stlink
that referenced
this issue
Sep 30, 2024
- Added '-V' and '-F' option to the getopt_long() optstring.
- Marked -F/--freq option with required argument.
- Added NULL pointer checks to helper function arg_parse_freq().
Nightwalker-87
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-V and -F options are not recognized on the command line of st-util program. When using --freq option a segmentation fault is the result.
examples:
$st-util -p 3333 -V --connect-under-reset --multi --semihosting -v99
st-util: invalid option -- 'V'
$st-util -p 3333 --connect-under-reset --multi --semihosting -v99 -F 2M
st-util: invalid option -- 'F'
non-option ARGV-elements: 2M
$st-util -p 3333 --connect-under-reset --multi --semihosting -v99 --freq 500k
st-util: Speicherzugriffsfehler
The reason is a bug in a getopt_long() parameters in main() function and missing NULL pointer security in arg_parse_freq() helper function.
See PR #1429
The text was updated successfully, but these errors were encountered: