Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Mon,  4 Dec 2017 12:13:06 +0100
From: Jo-Philipp Wich <jo@...n.io>
To: musl@...ts.openwall.com
Cc: Jo-Philipp Wich <jo@...n.io>
Subject: [PATCH] nameser.h: explicitely include endian.h

As nameser.h itself uses endian macros to declare the proper layout of
the HEADER structure, it needs to include the endian.h header as well.

The issue was found while investigating iodine query packet corruption
on little endian musl systems [1].

1: http://lists.infradead.org/pipermail/lede-dev/2017-November/010085.html

Signed-off-by: Jo-Philipp Wich <jo@...n.io>
---
 include/arpa/nameser.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/arpa/nameser.h b/include/arpa/nameser.h
index 581925a4..b315e0f3 100644
--- a/include/arpa/nameser.h
+++ b/include/arpa/nameser.h
@@ -7,6 +7,7 @@ extern "C" {
 
 #include <stddef.h>
 #include <stdint.h>
+#include <endian.h>
 
 #define __NAMESER	19991006
 #define NS_PACKETSZ	512
-- 
2.11.0

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.