openwrt/flashrom
1
0
Fork 0
mirror of https://review.coreboot.org/flashrom.git synced 2025-04-28 07:23:43 +02:00
Code Issues Releases Wiki Activity

layout.c: Cast argument of isspace() to unsigned char

Browse Source 
These functions [isspace, etc] check whether c [(the argument)], which
must have the value of an unsigned char or EOF, falls into a certain
character class according to the current locale.

Cast the argument from `char` to `unsigned char` to guarantee that
we don't pass illegal negative values. Some implementations actually
provide a warning to get heads up[1].

[1] https://man.netbsd.org/ctype.3#CAVEATS

TEST=Builds on Cygwin (Windows 10, amd64, gcc 11.3.0)

Change-Id: Ia48d5a19b0964bc28e5360edf06bdf287dad5945
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/66548
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
This commit is contained in:
Thomas Heijligen 2022-08-08 22:19:33 +02:00 committed by Thomas Heijligen
parent 650f082431
commit b6f3c00fbd
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
layout.c
View File

@ -312,7 +312,7 @@ void prepare_layout_for_extraction(struct flashctx *flash)
entry->file = strdup(entry->name); entry->file = strdup(entry->name);
for (i = 0; entry->file[i]; ++i) { for (i = 0; entry->file[i]; ++i) {
if (isspace(entry->file[i])) if (isspace((unsigned char)entry->file[i]))
entry->file[i] = '_'; entry->file[i] = '_';
} }
} }