mm: fix vm_brk_flags() to not bail out while holding lock
authorSebastian Ott <sebott@redhat.com>
Fri, 29 Sep 2023 17:19:41 +0000 (10:19 -0700)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 6 Oct 2023 21:11:38 +0000 (14:11 -0700)
commite0f81ab1e4f42ffece6440dc78f583eb352b9a71
treee1dd4a6ade832ee3794efcfd2f6713e6d0863b20
parent51f625377561e5b167da2db5aafb7ee268f691c5
mm: fix vm_brk_flags() to not bail out while holding lock

Calling vm_brk_flags() with flags set other than VM_EXEC will exit the
function without releasing the mmap_write_lock.

Just do the sanity check before the lock is acquired.  This doesn't fix an
actual issue since no caller sets a flag other than VM_EXEC.

Link: https://lkml.kernel.org/r/20230929171937.work.697-kees@kernel.org
Fixes: 2e7ce7d354f2 ("mm/mmap: change do_brk_flags() to expand existing VMA and add do_brk_munmap()")
Signed-off-by: Sebastian Ott <sebott@redhat.com>
Signed-off-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Liam R. Howlett <Liam.Howlett@oracle.com>
Cc: Yu Zhao <yuzhao@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/mmap.c