Bug in aclocal.m4 / configure ? BASH_STRUCT_WEXITSTATUS_OFFSET

[Andreas K . Hüttel]

Hi, 

in aclocal.m4 (and thus also configure), the test for 
BASH_STRUCT_WEXITSTATUS_OFFSET contains

  /* crack s */
  for (i = 0; i < (sizeof(s) - 8); i++)

Shouldnt this be 

  /* crack s */
  for (i = 0; i < (sizeof(s) * 8); i++)

(* instead of -), see attached trivial patch? The minus sign makes no sense at 
all since even on x86-64 sizeof(int)=4 ...

[I found this when tracking down weird behaviour in a highly experimental 
riscv32 / glibc / qemu chroot, where s=10240 ... that's a different bug in 
another moving piece though...]

Best,
Andreas

-- 
Andreas K. Hüttel
dilfridge@gentoo.org
Gentoo Linux developer 
(council, qa, toolchain, base-system, perl, libreoffice)

0001-Fix-typo-8-instead-of-8-when-testing-for-bit-shift.patch
Description: Text Data

signature.asc
Description: This is a digitally signed message part.