Whamcloud - gitweb
LU-16510 build: check if CONFIG_FORTIFY_SOURCE is defined 73/51973/2
authorJian Yu <yujian@whamcloud.com>
Thu, 17 Aug 2023 20:46:38 +0000 (13:46 -0700)
committerOleg Drokin <green@whamcloud.com>
Thu, 31 Aug 2023 06:39:53 +0000 (06:39 +0000)
The linux/fortify-string.h header file should not be
included while the kernel config option CONFIG_FORTIFY_SOURCE
is not defined.

Change-Id: I2e1905406e892b182f143d512a2d3722b141e52d
Fixes: 919b93b951d4 ("LU-16510 build: fortified memcpy from linux 6.1")
Signed-off-by: Jian Yu <yujian@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51973
Reviewed-by: Patrick Farrell <pfarrell@whamcloud.com>
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
libcfs/include/libcfs/linux/linux-fortify-string.h

index b00d9f7..b8ec03e 100644 (file)
@@ -1,7 +1,9 @@
 #ifndef _LIBCFS_FORTIFY_STRING_H
 #define _LIBCFS_FORTIFY_STRING_H
 
-#ifdef HAVE_LINUX_FORTIFY_STRING_HEADER
+#if defined(HAVE_LINUX_FORTIFY_STRING_HEADER) && \
+       !defined(__NO_FORTIFY) && defined(__OPTIMIZE__) && \
+       defined(CONFIG_FORTIFY_SOURCE)
 #include <linux/fortify-string.h>
 
 /*