broken kernel builds due to hexdump broken in busybox 1.37.0

2024-10-23 Thread Natanael Copa
Hi! The linux (efi) kernel in alpine edge is currently broken. We have traced it to busybox hexdump. # util-linux: ncopa-desktop:~/tmp$ echo "A" > test ncopa-desktop:~/tmp$ echo $(hexdump -s16 -n4 -e '"%u"' test) 1094795585 ncopa-desktop:~/tmp$ ec

Re: [PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Natanael Copa
On Wed, 23 Oct 2024 15:11:20 +0200 Natanael Copa wrote: > Add test that uncovers bug introduced in busybox 1.37.0 > --- > testsuite/hexdump.tests | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/testsuite/hexdump.tests b/testsuite/hexdump.tests > index be0379cfc..b933cd1a8 100755

[PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Natanael Copa
Add test that uncovers bug introduced in busybox 1.37.0 --- testsuite/hexdump.tests | 7 +++ 1 file changed, 7 insertions(+) diff --git a/testsuite/hexdump.tests b/testsuite/hexdump.tests index be0379cfc..b933cd1a8 100755 --- a/testsuite/hexdump.tests +++ b/testsuite/hexdump.tests @@ -82,4 +8

Re: [PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Clayton Craft
On Wed, 23 Oct 2024 15:11:20 +0200 Natanael Copa wrote: > I git bisected the bug to: > commit e2287f99fe6f21fd6435ad04340170ad4ba5f6b3 I found that if this one byte is reverted from the patch you blamed, then your test passes: diff --git a/libbb/dump.c b/libbb/dump.c index b406a2428..92b0d762c

Re: [PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Natanael Copa
I can confirm this. The tests for od passes as well. Denys, maybe this can be applied? Natanael Copa ons. 23. okt. 2024 kl. 20:12 skrev Clayton Craft : > On Wed, 23 Oct 2024 15:11:20 +0200 Natanael Copa > wrote: > > I git bisected the bug to: > > commit e2287f99fe6f21fd6435ad04340170ad4ba5f6b

Re: [PATCH 1/1] ping: Add support for IPv4-mapped IPv6 address format

2024-10-23 Thread Steffen Nurpmeso
Steffen Nurpmeso wrote in <20241023211752.4ZEO6ssL@steffen%sdaoden.eu>: ... |On the other hand Linux (and OpenIndiana/Solaris) just (seem to) |work neatlessly with IPv4-mapped addresses (for server sockets). To restrict this, this was TCP. --steffen | |Der Kragenbaer,The moon

Re: [PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Steffen Nurpmeso
Natanael Copa wrote in <20241023152936.658a4fdd@ncopa-desktop>: |On Wed, 23 Oct 2024 15:11:20 +0200 |Natanael Copa wrote: | |> Add test that uncovers bug introduced in busybox 1.37.0 |> --- |> testsuite/hexdump.tests | 7 +++ |> 1 file changed, 7 insertions(+) |> |> diff --git a/te

Re: Re: [PATCH 1/1] ping: Add support for IPv4-mapped IPv6 address format

2024-10-23 Thread Vo Ya
> Based on idea contributed to iputils. > https://github.com/iputils/iputils/commit/8ed7ffc999e2a541e06ee48faf26a323dfe487c2 Comparing some linux ping implementations that used glibc getaddrinfo() which already converts IPv4-mapped address string to IPv4 struct for requested AF_INET family In case

Re: [PATCH 1/1] ping: Add support for IPv4-mapped IPv6 address format

2024-10-23 Thread Steffen Nurpmeso
David Leonard wrote in <1qs786o8-714q-23nn-649q-o331s404s...@nqncgvir-ragrecevfrf.pbz>: |On Sun, 20 Oct 2024, Lars Uffmann wrote: |> As for the points David brings up: |> On 2024-10-20 13:51:32, Petr Vorel wrote: | I'm not a fan of this patch because ping is a low-level diagnostic \ |

Re: [PATCH] hexdump: add test to verify -n4 -e '"%u"'

2024-10-23 Thread Steffen Nurpmeso
Steffen Nurpmeso wrote in <20241023213552.tNfZc0Ju@steffen%sdaoden.eu>: |Natanael Copa wrote in | <20241023152936.658a4fdd@ncopa-desktop>: ||On Wed, 23 Oct 2024 15:11:20 +0200 ||Natanael Copa wrote: || ||> Add test that uncovers bug introduced in busybox 1.37.0 ... ||I git bisected the bu