coja/lbmk
1
0
Fork 0
mirror of https://codeberg.org/libreboot/lbmk.git synced 2026-07-21 19:26:22 +02:00
Code Issues Packages Projects Releases Wiki Activity

util/nvmutil: tidy up conv_argv_part_num

Browse Source 
no need for strlen here (1-character string).

Signed-off-by: Leah Rowe <leah@libreboot.org>
This commit is contained in:
Leah Rowe
2026-03-07 00:08:17 +00:00
parent 330b2da20c
commit 658ed55bc1
1 changed files with 3 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
util/nvmutil/nvmutil.c
+3 -11
View File
@@ -277,9 +277,6 @@ check_cmd_args(int argc, char *argv[])
err(EINVAL, "Bad command"); err(EINVAL, "Bad command");
} }
/*
* Not to be confused with check_part_num()
*/
static size_t static size_t
conv_argv_part_num(const char *part_str) conv_argv_part_num(const char *part_str)
{ {
@@ -290,12 +287,10 @@ conv_argv_part_num(const char *part_str)
* it is assumed to be signed, and handled accordingly. * it is assumed to be signed, and handled accordingly.
*/ */
if (strlen(part_str) != 1) if (part_str[0] == '\0' || part_str[1] != '\0')
err(EINVAL, "Partnum string '%s' wrong length.", err(EINVAL, "Partnum string '%s' wrong length.", part_str);
part_str);
ch = (unsigned char)part_str[0]; ch = (unsigned char)part_str[0] - '0';
ch -= '0';
check_part_num((size_t)ch); check_part_num((size_t)ch);
return (size_t)ch; return (size_t)ch;
@@ -887,9 +882,6 @@ set_part_modified(size_t p)
part_modified[p] = 1; part_modified[p] = 1;
} }
/*
* Not to be confused with conv_argv_part_num()
*/
static void static void
check_part_num(size_t p) check_part_num(size_t p)
{ {