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: cec-ctl: use strtoul instead of strtol where appropriate Author: Hans Verkuil <[email protected]> Date: Fri Sep 26 10:23:34 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/cec-ctl/cec-ctl.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) --- http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=1589ecd04cfd7c89fd0c990c527cb665db9047bf diff --git a/utils/cec-ctl/cec-ctl.cpp b/utils/cec-ctl/cec-ctl.cpp index 3241baac92d0..3e6454a591e5 100644 --- a/utils/cec-ctl/cec-ctl.cpp +++ b/utils/cec-ctl/cec-ctl.cpp @@ -2643,7 +2643,7 @@ int main(int argc, char **argv) switch (cec_parse_subopt(&subs, arg_names, &value)) { case 0: while (size < sizeof(bytes)) { - bytes[size++] = strtol(value, &endptr, 0L); + bytes[size++] = strtoul(value, &endptr, 0L); if (endptr == value) { size--; break; @@ -2679,12 +2679,12 @@ int main(int argc, char **argv) while (*subs != '\0') { switch (cec_parse_subopt(&subs, arg_names, &value)) { case 0: - cmd = strtol(value, &endptr, 0L); + cmd = strtoul(value, &endptr, 0L); have_cmd = true; break; case 1: while (size < sizeof(bytes)) { - bytes[size++] = strtol(value, &endptr, 0L); + bytes[size++] = strtoul(value, &endptr, 0L); if (endptr == value) { size--; break; @@ -2722,15 +2722,15 @@ int main(int argc, char **argv) while (*subs != '\0') { switch (cec_parse_subopt(&subs, arg_names, &value)) { case 0: - vendor_id = strtol(value, nullptr, 0); + vendor_id = strtoul(value, nullptr, 0); break; case 1: - msg.reply = strtol(value, &endptr, 0L); + msg.reply = strtoul(value, &endptr, 0L); msg.flags = CEC_MSG_FL_REPLY_VENDOR_ID; break; case 2: while (size < sizeof(bytes)) { - bytes[size++] = strtol(value, &endptr, 0L); + bytes[size++] = strtoul(value, &endptr, 0L); if (endptr == value) { size--; break; @@ -2764,7 +2764,7 @@ int main(int argc, char **argv) switch (cec_parse_subopt(&subs, arg_names, &value)) { case 0: while (size < sizeof(bytes)) { - bytes[size++] = strtol(value, &endptr, 0L); + bytes[size++] = strtoul(value, &endptr, 0L); if (endptr == value) { size--; break;
