diff mbox series

[1/1] lapi/fsmount: Further fix header conflict on old distros

Message ID 20240117195432.2719142-1-pvorel@suse.cz
State Accepted
Headers show
Series [1/1] lapi/fsmount: Further fix header conflict on old distros | expand

Commit Message

Petr Vorel Jan. 17, 2024, 7:54 p.m. UTC
12867c0d6 fix on lapi/io_uring was not enough.  glibc doc [1] mentions
conflict between <linux/mount.h> and <sys/mount.h>. Because <linux/fs.h>
includes <linux/mount.h> it's easy to get <linux/mount.h> mounted
indirectly.

Problem was fixed on glibc 2.37:
774058d729 ("linux: Fix sys/mount.h usage with kernel headers")
and it got backported to git tree for 2.36 and to some distros.
But it still fails for older glibc.

Therefore include <sys/mount.h> only if <linux/mount.h> not already
included.

[1] https://sourceware.org/glibc/wiki/Synchronizing_Headers
Fixes: #1121
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
CI: https://github.com/pevik/ltp/actions/runs/7561031650

 include/lapi/fsmount.h | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

Comments

Petr Vorel Jan. 17, 2024, 8:51 p.m. UTC | #1
Hi all,

merged to fix CI.

Kind regards,
Petr
diff mbox series

Patch

diff --git a/include/lapi/fsmount.h b/include/lapi/fsmount.h
index 07eb42ffa..2ed4da86f 100644
--- a/include/lapi/fsmount.h
+++ b/include/lapi/fsmount.h
@@ -11,12 +11,11 @@ 
 #include "config.h"
 #include <sys/syscall.h>
 #include <sys/types.h>
-#include <sys/mount.h>
 
-#ifndef HAVE_FSOPEN
-# ifdef HAVE_LINUX_MOUNT_H
-#  include <linux/mount.h>
-# endif
+#if !defined(HAVE_FSOPEN) && defined(HAVE_LINUX_MOUNT_H)
+# include <linux/mount.h>
+#else
+# include <sys/mount.h>
 #endif
 
 #include "lapi/fcntl.h"