This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:
Subject: ivtv-ctl: use strtoul instead of strtol where appropriate Author: Hans Verkuil <[email protected]> Date: Fri Sep 26 10:23:12 2025 +0200 In several cases command line arguments were parsed using strtol, even though the argument can never be negative. Change it to strtoul. Signed-off-by: Hans Verkuil <[email protected]> utils/ivtv-ctl/ivtv-ctl.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=2e4f05d43bc0b485f29b411d13cc7d103ce717d7 diff --git a/utils/ivtv-ctl/ivtv-ctl.c b/utils/ivtv-ctl/ivtv-ctl.c index bb9374542a9b..19bddfdbcc59 100644 --- a/utils/ivtv-ctl/ivtv-ctl.c +++ b/utils/ivtv-ctl/ivtv-ctl.c @@ -326,8 +326,8 @@ int main(int argc, char **argv) return 1; } - yuv_mode = strtol(value, 0L, 0); - if (yuv_mode < 0 || yuv_mode > 3) { + yuv_mode = strtoul(value, 0L, 0); + if (yuv_mode > 3) { fprintf(stderr, "invalid yuv mode\n"); return 1; } @@ -340,7 +340,7 @@ int main(int argc, char **argv) usage(); return 0; case OptSetDebugLevel:{ - debug_level = strtol(optarg, 0L, 0); + debug_level = strtoul(optarg, 0L, 0); break; } case OptSetDevice: @@ -353,10 +353,10 @@ int main(int argc, char **argv) } break; case OptReset: - reset = strtol(optarg, 0L, 0); + reset = strtoul(optarg, 0L, 0); break; case OptPassThrough: - passthrough = strtol(optarg, 0L, 0); + passthrough = strtoul(optarg, 0L, 0); break; case OptSetGPIO: subs = optarg; @@ -379,7 +379,7 @@ int main(int argc, char **argv) usage(); exit(1); } - gpio_out = (unsigned short)strtol(value, 0L, 0); + gpio_out = strtoul(value, 0L, 0); break; default: fprintf(stderr,
