iodine: workaround incomplete musl nameser.h header 5190/head
authorJo-Philipp Wich <jo@mein.io>
Wed, 29 Nov 2017 17:13:36 +0000 (18:13 +0100)
committerJo-Philipp Wich <jo@mein.io>
Tue, 12 Dec 2017 22:41:44 +0000 (23:41 +0100)
commit175cf9e985ff27d8a8561a4304e2bb63449b9cb1
tree8de87d1a3ddb3020c906fe30e205ef0b257cf4ff
parent70858690c08773c5e420f9ec66ed95c20b593212
iodine: workaround incomplete musl nameser.h header

The arpa/nameser.h header of musl libc indirectly depends on the endian.h
header but fails to explicitely include it to properly define
`__BYTE_ORDER` and `__BIG_ENDIAN` prior to declaring the DNS `HEADER`
structure.

When both the appropriate `__BYTE_ORDER` and `__BIG_ENDIAN` defines are
unset, the `#if __BYTE_ORDER == __BIG_ENDIAN` condition in `nameser.h`
evaluates to true, causing it to declare a bad (big endian) DNS packet
header structure on little endian systems.

Work around this musl bug by forcibly passing `-include endian.h` through
the `osflags` file.

An upstream fix for musl libc has been submitted with
http://www.openwall.com/lists/musl/2017/12/04/3

This should solve iodine packet corruption on little endian musl systems
reported at
http://lists.infradead.org/pipermail/lede-dev/2017-November/010085.html

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
net/iodine/Makefile
net/iodine/patches/101-musl-workaround-incomplete-nameser-h.patch [new file with mode: 0644]