On 14/05/2019 20.04, Christian Borntraeger wrote: > > > On 14.05.19 20:03, Markus Armbruster wrote: >> 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 <borntrae...@de.ibm.com> > Acked-by: Christian Borntraeger <borntrae...@de.ibm.com>
FYI, this patch is already queued in Cornelia's s390-next tree (and it should also go via her tree, since you need to rebuild the bios binary afterwards). Thomas