[<prev] [next>] [day] [month] [year] [list]
Message-ID: <174221144059.14745.8349031235521776012.tip-bot2@tip-bot2>
Date: Mon, 17 Mar 2025 11:37:20 -0000
From: "tip-bot2 for Ingo Molnar" <tip-bot2@...utronix.de>
To: linux-tip-commits@...r.kernel.org
Cc: Ingo Molnar <mingo@...nel.org>, Josh Poimboeuf <jpoimboe@...hat.com>,
Peter Zijlstra <peterz@...radead.org>, linux-kernel@...r.kernel.org,
x86@...nel.org
Subject: [tip: objtool/core] objtool: Use O_CREAT with explicit mode mask
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@...nel.org>
AuthorDate: Mon, 17 Mar 2025 12:21:57 +01:00
Committer: Ingo Molnar <mingo@...nel.org>
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@...nel.org>
Cc: Josh Poimboeuf <jpoimboe@...hat.com>
Cc: Peter Zijlstra <peterz@...radead.org>
Cc: linux-kernel@...r.kernel.org
---
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;
Powered by blists - more mailing lists