The following commit has been merged into the objtool/core branch of tip: Commit-ID: 73070466ed3b5e4620e03c159ee12a570b171d08 Gitweb: https://git.kernel.org/tip/73070466ed3b5e4620e03c159ee12a570b171d08 Author: Ingo Molnar <mingo@xxxxxxxxxx> AuthorDate: Mon, 17 Mar 2025 12:21:57 +01:00 Committer: Ingo Molnar <mingo@xxxxxxxxxx> CommitterDate: Mon, 17 Mar 2025 12:24:02 +01:00 objtool: Use O_CREAT with explicit mode mask Recent Ubuntu enforces 3-argument open() with O_CREAT: CC /home/mingo/tip/tools/objtool/builtin-check.o In file included from /usr/include/fcntl.h:341, from builtin-check.c:9: In function ‘open’, inlined from ‘copy_file’ at builtin-check.c:201:11: /usr/include/x86_64-linux-gnu/bits/fcntl2.h:52:11: error: call to ‘__open_missing_mode’ declared with attribute error: open with O_CREAT or O_TMPFILE in second argument needs 3 arguments 52 | __open_missing_mode (); | ^~~~~~~~~~~~~~~~~~~~~~ Use 0400 as the most restrictive mode for the new file. Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx> Cc: Josh Poimboeuf <jpoimboe@xxxxxxxxxx> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx> Cc: linux-kernel@xxxxxxxxxxxxxxx --- tools/objtool/builtin-check.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/objtool/builtin-check.c b/tools/objtool/builtin-check.c index 39ddca6..5f761f4 100644 --- a/tools/objtool/builtin-check.c +++ b/tools/objtool/builtin-check.c @@ -198,7 +198,7 @@ static int copy_file(const char *src, const char *dst) return 1; } - dst_fd = open(dst, O_WRONLY | O_CREAT | O_TRUNC); + dst_fd = open(dst, O_WRONLY | O_CREAT | O_TRUNC, 0400); if (dst_fd == -1) { ERROR("can't open '%s' for writing", dst); return 1;
![]() |