This shouldn't change behavior in any way - at this point, there should be no non-writable mappings, only the initial stack mapping -, but this change makes it easier to reason about the correctness of the following commits that place restrictions on forced memory writes.
Signed-off-by: Jann Horn <[email protected]> Reviewed-by: Janis Danisevskis <[email protected]> --- fs/exec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/exec.c b/fs/exec.c index 6fcfb3f..d607da8 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -204,7 +204,7 @@ static struct page *get_arg_page(struct linux_binprm *bprm, unsigned long pos, * doing the exec and bprm->mm is the new process's mm. */ ret = get_user_pages_remote(current, bprm->mm, pos, 1, write, - 1, &page, NULL); + 0, &page, NULL); if (ret <= 0) return NULL; -- 2.1.4
