diff options
author | Fathi Boudra <fathi.boudra@linaro.org> | 2017-06-20 18:30:54 +0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-06-28 15:51:59 +0100 |
commit | 3c1d9629ac3b5af47d0f4db09b4eb7e0eb4b0afe (patch) | |
tree | a0f7d713ce70436d01b9931548573a076d194863 /meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch | |
parent | 2a0afa9682d6119f403626ca31cd8c9854637312 (diff) | |
download | openembedded-core-contrib-3c1d9629ac3b5af47d0f4db09b4eb7e0eb4b0afe.tar.gz |
ltp: use upstream patch to fix faccessat/fchmodat build warnings
Use 0037-faccessat-and-fchmodat-Fx-build-warnings.patch instead of
0012-fix-faccessat01.c-build-fails-with-security-flags.patch.
The new patch has been merged upstream and will be available in next LTP
release.
Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Diffstat (limited to 'meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch')
-rw-r--r-- | meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch | 68 |
1 files changed, 68 insertions, 0 deletions
diff --git a/meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch b/meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch new file mode 100644 index 0000000000..99497d655a --- /dev/null +++ b/meta/recipes-extended/ltp/ltp/0037-faccessat-and-fchmodat-Fix-build-warnings.patch @@ -0,0 +1,68 @@ +From 4141bdb441f7576a2e73abeb2a0fb1c7b8bd8647 Mon Sep 17 00:00:00 2001 +From: Naresh Kamboju <naresh.kamboju@linaro.org> +Date: Mon, 19 Jun 2017 19:01:21 +0530 +Subject: [PATCH 1/7] faccessat and fchmodat: Fix build warnings +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Fixing below build warnings. + +ltp-full-20170516/testcases/kernel/syscalls/faccessat$ make +In file included from ../../../../include/old/test.h:51:0, + from faccessat01.c:44: +faccessat01.c: In function ‘setup’: +../../../../include/old/old_safe_file_ops.h:55:27: +warning: format not a string literal and no format arguments [-Wformat-security] + (path), (fmt), ## __VA_ARGS__) + ^ +faccessat01.c:132:2: note: in expansion of macro ‘SAFE_FILE_PRINTF’ + SAFE_FILE_PRINTF(cleanup, testfile, testfile); + ^ +../../../../include/old/old_safe_file_ops.h:55:27: +warning: format not a string literal and no format arguments [-Wformat-security] + (path), (fmt), ## __VA_ARGS__) + ^ +faccessat01.c:133:2: note: in expansion of macro ‘SAFE_FILE_PRINTF’ + SAFE_FILE_PRINTF(cleanup, testfile2, testfile2); + +Signed-off-by: Naresh Kamboju <naresh.kamboju@linaro.org> +Upstream-Status: Accepted [https://github.com/linux-test-project/ltp/commit/4141bdb441f7576a2e73abeb2a0fb1c7b8bd8647] +--- + testcases/kernel/syscalls/faccessat/faccessat01.c | 4 ++-- + testcases/kernel/syscalls/fchmodat/fchmodat01.c | 4 ++-- + 2 files changed, 4 insertions(+), 4 deletions(-) + +diff --git a/testcases/kernel/syscalls/faccessat/faccessat01.c b/testcases/kernel/syscalls/faccessat/faccessat01.c +index 622dfd3b0..def7fa755 100644 +--- a/testcases/kernel/syscalls/faccessat/faccessat01.c ++++ b/testcases/kernel/syscalls/faccessat/faccessat01.c +@@ -129,8 +129,8 @@ void setup(void) + fds[0] = SAFE_OPEN(cleanup, pathname, O_DIRECTORY); + fds[1] = fds[4] = fds[0]; + +- SAFE_FILE_PRINTF(cleanup, testfile, testfile); +- SAFE_FILE_PRINTF(cleanup, testfile2, testfile2); ++ SAFE_FILE_PRINTF(cleanup, testfile, "%s", testfile); ++ SAFE_FILE_PRINTF(cleanup, testfile2, "%s", testfile2); + + fds[2] = SAFE_OPEN(cleanup, testfile3, O_CREAT | O_RDWR, 0600); + +diff --git a/testcases/kernel/syscalls/fchmodat/fchmodat01.c b/testcases/kernel/syscalls/fchmodat/fchmodat01.c +index 6bf66d8b8..7856ed3fa 100644 +--- a/testcases/kernel/syscalls/fchmodat/fchmodat01.c ++++ b/testcases/kernel/syscalls/fchmodat/fchmodat01.c +@@ -127,8 +127,8 @@ void setup(void) + fds[0] = SAFE_OPEN(cleanup, pathname, O_DIRECTORY); + fds[1] = fds[4] = fds[0]; + +- SAFE_FILE_PRINTF(cleanup, testfile, testfile); +- SAFE_FILE_PRINTF(cleanup, testfile2, testfile2); ++ SAFE_FILE_PRINTF(cleanup, testfile, "%s", testfile); ++ SAFE_FILE_PRINTF(cleanup, testfile2, "%s", testfile2); + + fds[2] = SAFE_OPEN(cleanup, testfile3, O_CREAT | O_RDWR, 0600); + fds[3] = 100; +-- +2.13.0 + |