[LTP] [PATCH] getdents: Fix build under glibc 2.30

Petr Vorel pvorel@suse.cz
Tue Oct 8 11:32:18 CEST 2019


glibc commit b8b3d5a14e ("Linux: Move getdents64 to <dirent.h>")
moved the declaration from <unistd.h> to <dirent.h> to match the
location of the declaration in musl.

Thus we need to include both <dirent.h> and <unistd.h> for getdents64().

Using getdents64() declaration requires on both glibc and musl
_GNU_SOURCE definition, thus move it to to getdents.h.

Fixes: 587
Reported-by: Cyril Hrubis <chrubis@suse.cz>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Hi,

you might not like moving _GNU_SOURCE definition to the header, I can
add the missing one in getdents01.c instead.

Travis: https://travis-ci.org/pevik/ltp/builds/595021543

Kind regards,
Petr

 testcases/kernel/syscalls/getdents/getdents.h   | 2 ++
 testcases/kernel/syscalls/getdents/getdents01.c | 3 ++-
 testcases/kernel/syscalls/getdents/getdents02.c | 4 ++--
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/testcases/kernel/syscalls/getdents/getdents.h b/testcases/kernel/syscalls/getdents/getdents.h
index c24ed6c99..e43dacc15 100644
--- a/testcases/kernel/syscalls/getdents/getdents.h
+++ b/testcases/kernel/syscalls/getdents/getdents.h
@@ -20,6 +20,7 @@
 #ifndef GETDENTS_H
 #define GETDENTS_H
 
+#define _GNU_SOURCE
 #include <stdint.h>
 #include "test.h"
 #include "lapi/syscalls.h"
@@ -54,6 +55,7 @@ struct linux_dirent64 {
 };
 
 #if HAVE_GETDENTS64
+#include <dirent.h>
 #include <unistd.h>
 #else
 static inline int
diff --git a/testcases/kernel/syscalls/getdents/getdents01.c b/testcases/kernel/syscalls/getdents/getdents01.c
index 3962d960b..17a58731d 100644
--- a/testcases/kernel/syscalls/getdents/getdents01.c
+++ b/testcases/kernel/syscalls/getdents/getdents01.c
@@ -19,6 +19,8 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#include "getdents.h"
+
 #include <stdio.h>
 #include <errno.h>
 #include <sys/types.h>
@@ -27,7 +29,6 @@
 
 #include "test.h"
 #include "safe_macros.h"
-#include "getdents.h"
 
 static void cleanup(void);
 static void setup(void);
diff --git a/testcases/kernel/syscalls/getdents/getdents02.c b/testcases/kernel/syscalls/getdents/getdents02.c
index 7b023c53f..c45812241 100644
--- a/testcases/kernel/syscalls/getdents/getdents02.c
+++ b/testcases/kernel/syscalls/getdents/getdents02.c
@@ -33,7 +33,8 @@
  *
  */
 
-#define _GNU_SOURCE
+#include "getdents.h"
+
 #include <stdio.h>
 #include <errno.h>
 #include <sys/types.h>
@@ -41,7 +42,6 @@
 #include <fcntl.h>
 
 #include "test.h"
-#include "getdents.h"
 #include "safe_macros.h"
 
 #define DIR_MODE	(S_IRUSR|S_IWUSR|S_IXUSR|S_IRGRP| \
-- 
2.23.0



More information about the ltp mailing list