umockdev icon indicating copy to clipboard operation
umockdev copied to clipboard

pcap: Add appropriate endianness conversions for USB pcap header

Open benzea opened this issue 1 year ago • 4 comments

The USB pcap header should always be little endian. As such, add the appropriate conversions where necessary to ensure that the data is interpreted correctly on big endian architectures.

Maybe this fixes the sparc SIGBUS problem

benzea avatar Mar 03 '24 11:03 benzea

Hmm, that amd64 ubuntu:devel failure does not look like a regression from this patch (failing after umockdev-testbed/libc so /umockdev-testbed/uevent/libudev). Looks a little bit odd to be honest …

benzea avatar Mar 03 '24 12:03 benzea

Uh, did the test previously pass on s390x? I am a bit confused that the urb is not reaped in https://download.copr.fedorainfracloud.org/results/packit/martinpitt-umockdev-234/fedora-rawhide-s390x/07099222-umockdev/builder-live.log.gz :-)

But, I think I would need to debug on an s390x machine. Just staring at the code I don't see anything wrong.

benzea avatar Mar 03 '24 12:03 benzea

that amd64 ubuntu:devel failure does not look like a regression from this patch

I haven't seen it fail recently, but I retried and it passed.

Uh, did the test previously pass on s390x?

Yes, it did. Everything has been green for a while except for nix, and that got fixed this morning. So this is definitively a regression. Note that s390x is big-endian (for that very reason I want it in PR tests).

martinpitt avatar Mar 03 '24 16:03 martinpitt

Yes, it did. Everything has been green for a while except for nix, and that got fixed this morning. So this is definitively a regression. Note that s390x is big-endian (for that very reason I want it in PR tests).

That is … odd. It must be pure luck that it is working, but no idea right now how that could happen (possibly because those are mostly control transfers). Anyway, later.

benzea avatar Mar 04 '24 07:03 benzea