Skip to content

Commit cd57e44

Browse files
committed
exec: Remove FOLL_FORCE for stack setup
It does not appear that FOLL_FORCE should be needed for setting up the stack pages. They are allocated using the nascent brpm->vma, which was newly created with VM_STACK_FLAGS, which an arch can override, but they all appear to include VM_WRITE | VM_MAYWRITE. Remove FOLL_FORCE. Cc: Eric Biederman <[email protected]> Cc: David Hildenbrand <[email protected]> Cc: Linus Torvalds <[email protected]> Cc: Alexander Viro <[email protected]> Cc: [email protected] Cc: [email protected] Link: https://lore.kernel.org/lkml/202211171439.CDE720EAD@keescook/ Signed-off-by: Kees Cook <[email protected]>
1 parent dc64cc1 commit cd57e44

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

fs/exec.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ static struct page *get_arg_page(struct linux_binprm *bprm, unsigned long pos,
200200
{
201201
struct page *page;
202202
int ret;
203-
unsigned int gup_flags = FOLL_FORCE;
203+
unsigned int gup_flags = 0;
204204

205205
#ifdef CONFIG_STACK_GROWSUP
206206
if (write) {

0 commit comments

Comments
 (0)