[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 5/6] pc-bios/s390-ccw: Clean up harmless misuse o
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PATCH v2 5/6] pc-bios/s390-ccw: Clean up harmless misuse of isdigit() |
Date: |
Tue, 14 May 2019 20:03:10 +0200 |
atoui() and get_index() pass char values to isdigit(). With a
standard isdigit(), we'd get undefined behavior when the value is
negative. Can't happen as char is unsigned on s390x. Even if it
could, we're actually using isdigit() from pc-bios/s390-ccw/libc.h
here, which works fine for negative values. Clean up anyway, just
to avoid setting a bad example.
Cc: Christian Borntraeger <address@hidden>
Cc: Thomas Huth <address@hidden>
Cc: Cornelia Huck <address@hidden>
Cc: address@hidden
Signed-off-by: Markus Armbruster <address@hidden>
---
pc-bios/s390-ccw/libc.c | 2 +-
pc-bios/s390-ccw/menu.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/pc-bios/s390-ccw/libc.c b/pc-bios/s390-ccw/libc.c
index a786566c4c..3187923950 100644
--- a/pc-bios/s390-ccw/libc.c
+++ b/pc-bios/s390-ccw/libc.c
@@ -38,7 +38,7 @@ uint64_t atoui(const char *str)
}
while (*str) {
- if (!isdigit(*str)) {
+ if (!isdigit(*(unsigned char *)str)) {
break;
}
val = val * 10 + *str - '0';
diff --git a/pc-bios/s390-ccw/menu.c b/pc-bios/s390-ccw/menu.c
index 82a4ae6315..ce3815b201 100644
--- a/pc-bios/s390-ccw/menu.c
+++ b/pc-bios/s390-ccw/menu.c
@@ -134,7 +134,7 @@ static int get_index(void)
/* Check for erroneous input */
for (i = 0; i < len; i++) {
- if (!isdigit(buf[i])) {
+ if (!isdigit((unsigned char)buf[i])) {
return -1;
}
}
--
2.17.2
- [Qemu-devel] [PATCH v2 0/6] Fix misuse of ctype.h functions, Markus Armbruster, 2019/05/14
- [Qemu-devel] [PATCH v2 1/6] qemu-bridge-helper: Fix misuse of isspace(), Markus Armbruster, 2019/05/14
- [Qemu-devel] [PATCH v2 3/6] gdbstub: Reject invalid RLE repeat counts, Markus Armbruster, 2019/05/14
- [Qemu-devel] [PATCH v2 5/6] pc-bios/s390-ccw: Clean up harmless misuse of isdigit(),
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 6/6] cutils: Simplify how parse_uint() checks for whitespace, Markus Armbruster, 2019/05/14
- Re: [Qemu-devel] [PATCH v2 0/6] Fix misuse of ctype.h functions, Markus Armbruster, 2019/05/22