diff options
| author | Mirko Faina <mroik@delayed.space> | 2026-03-16 01:51:16 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2026-03-16 12:54:47 -0700 |
| commit | d05d84c5f507e8b973982e9cf3a27a07cd94fcb8 (patch) | |
| tree | a17f3e1b7f52a20e30dde86eea648eafaf6793d0 /tools | |
| parent | 67ad42147a7acc2af6074753ebd03d904476118f (diff) | |
| download | git-d05d84c5f507e8b973982e9cf3a27a07cd94fcb8.tar.xz | |
apply.c: fix -p argument parsing
"git apply" has an option -p that takes an integer as its argument.
Unfortunately the function apply_option_parse_p() in charge of parsing
this argument uses atoi() to convert from string to integer, which
allows a non-digit after the number (e.g. "1q") to be silently ignored.
As a consequence, an argument that does not begin with a digit silently
becomes a zero. Despite this command working fine when a non-positive
argument is passed, it might be useful for the end user to know that
their input contains non-digits that might've been unintended.
Replace atoi() with strtol_i() to catch malformed inputs.
Signed-off-by: Mirko Faina <mroik@delayed.space>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions
