[PATCH 1/1] sandbox: replace putchar(ch) by fputc(ch, stdout)

Simon Glass sjg at chromium.org
Thu Dec 2 22:18:12 CET 2021


On Sat, 20 Nov 2021 at 05:28, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> When compiled with -Og for better debugability u-boot ends up in a stack
> overflow using
>
>     gcc (Ubuntu 11.2.0-7ubuntu2) 11.2.0
>     GNU Binutils for Ubuntu 2.37
>
> putchar(ch) is defined as a macro which ends up calling U-Boot's putc()
> implementation instead of the glibc one, which calls os_putc() ...
>
> Let's use fputc(ch, stdout) instead as fputc() does not exist in U-Boot.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  arch/sandbox/cpu/os.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot-dm/next, thanks!


More information about the U-Boot mailing list