This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
[PATCH v2] Include sys/types.h before fts.h
- From: Ulf Hermann <ulf dot hermann at qt dot io>
- To: <elfutils-devel at sourceware dot org>
- Date: Thu, 20 Apr 2017 16:45:51 +0200
- Subject: [PATCH v2] Include sys/types.h before fts.h
- Authentication-results: sourceware.org; auth=none
- Authentication-results: sourceware.org; dkim=none (message not signed) header.d=none;sourceware.org; dmarc=none action=none header.from=qt.io;
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qtcompany.onmicrosoft.com; s=selector1-qt-io; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=RKsM/2sdlTiyqWl8VrD1vSsov1C7ZBsxd2OPx7ruzrA=; b=fCjVvrO/kS1XLXj0M049E2FkFsu25DXhjLyhjGNWqiHQKP0bHhD7qgfmEFV7XV3dPC2/XjimHUQlKW/MS8HYUGL6rDbJVbpnNt+4s7KO2N2ymj1z1gD609SHH8s/r9LJp5OtdvcHrjvvJ0gK1FNsXVxU2le8dBnUrlbc7XlCuUo=
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
The bad fts not only needs to be included before config.h, but also
requires various special types without including sys/types.h.
Change-Id: I31ac8d2aadcf7ffb3efb63583b2745991bfd6f90
Signed-off-by: Ulf Hermann <ulf.hermann@qt.io>
---
libdwfl/ChangeLog | 4 ++++
libdwfl/linux-kernel-modules.c | 6 +++++-
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/libdwfl/ChangeLog b/libdwfl/ChangeLog
index ede6d47..cc95e06 100644
--- a/libdwfl/ChangeLog
+++ b/libdwfl/ChangeLog
@@ -1,3 +1,7 @@
+2017-04-20 Ulf Hermann <ulf.hermann@qt.io>
+
+ * linux-kernel-modules.c: Include sys/types.h before fts.h
+
2017-03-24 Mark Wielaard <mark@klomp.org>
* linux-core-attach.c (core_next_thread): If n_namesz == 0 then
diff --git a/libdwfl/linux-kernel-modules.c b/libdwfl/linux-kernel-modules.c
index 7345e76..893110a 100644
--- a/libdwfl/linux-kernel-modules.c
+++ b/libdwfl/linux-kernel-modules.c
@@ -28,8 +28,11 @@
/* In case we have a bad fts we include this before config.h because it
can't handle _FILE_OFFSET_BITS.
- Everything we need here is fine if its declarations just come first. */
+ Everything we need here is fine if its declarations just come first.
+ Also, include sys/types.h before fts. On some systems fts.h is not self
+ contained. */
#ifdef BAD_FTS
+ #include <sys/types.h>
#include <fts.h>
#endif
@@ -55,6 +58,7 @@
#define fopen fopen64
#endif
#else
+ #include <sys/types.h>
#include <fts.h>
#endif
--
2.1.4