Files
kernel_arpi/mm
Suren Baghdasaryan 9b71fb2a76 UPSTREAM: mm: fix use-after-free bug when mm->mmap is reused after being freed
65;7000;1coom reaping (__oom_reap_task_mm) relies on a 2 way synchronization with
exit_mmap.  First it relies on the mmap_lock to exclude from unlock
path[1], page tables tear down (free_pgtables) and vma destruction.
This alone is not sufficient because mm->mmap is never reset.

For historical reasons[2] the lock is taken there is also MMF_OOM_SKIP
set for oom victims before.

The oom reaper only ever looks at oom victims so the whole scheme works
properly but process_mrelease can opearate on any task (with fatal
signals pending) which doesn't really imply oom victims.  That means
that the MMF_OOM_SKIP part of the synchronization doesn't work and it
can see a task after the whole address space has been demolished and
traverse an already released mm->mmap list.  This leads to use after
free as properly caught up by KASAN report.

Fix the issue by reseting mm->mmap so that MMF_OOM_SKIP synchronization
is not needed anymore.  The MMF_OOM_SKIP is not removed from exit_mmap
yet but it acts mostly as an optimization now.

[1] 27ae357fa8 ("mm, oom: fix concurrent munlock and oom reaper unmap, v3")
[2] 2129258024 ("mm: oom: let oom_reap_task and exit_mmap run concurrently")

[mhocko@suse.com: changelog rewrite]

Bug: 254441685
Link: https://lore.kernel.org/all/00000000000072ef2c05d7f81950@google.com/
Link: https://lkml.kernel.org/r/20220215201922.1908156-1-surenb@google.com
Fixes: 64591e8605d6 ("mm: protect free_pgtables with mmap_lock write lock in exit_mmap")
Signed-off-by: Suren Baghdasaryan <surenb@google.com>
Reported-by: syzbot+2ccf63a4bd07cf39cab0@syzkaller.appspotmail.com
Suggested-by: Michal Hocko <mhocko@suse.com>
Reviewed-by: Rik van Riel <riel@surriel.com>
Reviewed-by: Yang Shi <shy828301@gmail.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Cc: David Rientjes <rientjes@google.com>
Cc: Matthew Wilcox <willy@infradead.org>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Roman Gushchin <roman.gushchin@linux.dev>
Cc: Rik van Riel <riel@surriel.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Kirill A. Shutemov <kirill@shutemov.name>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Cc: Christian Brauner <brauner@kernel.org>
Cc: Christoph Hellwig <hch@infradead.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: David Hildenbrand <david@redhat.com>
Cc: Jann Horn <jannh@google.com>
Cc: Shakeel Butt <shakeelb@google.com>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: Christian Brauner <christian.brauner@ubuntu.com>
Cc: Florian Weimer <fweimer@redhat.com>
Cc: Jan Engelhardt <jengelh@inai.de>
Cc: Tim Murray <timmurray@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
(cherry picked from commit f798a1d4f94de9510e060d37b9b47721065a957c)
Signed-off-by: Lee Jones <joneslee@google.com>
Change-Id: I84fdfe87bab3df5e0ee6f1444e606b353f2e4662
2022-12-19 16:30:28 +00:00
..
2022-11-16 10:04:35 +00:00
2022-06-09 10:28:01 +02:00
2022-07-13 11:40:42 +02:00
2022-12-08 16:43:35 +00:00
2022-03-23 11:32:12 -07:00
2022-11-28 16:08:50 +00:00
2022-12-14 11:37:20 +01:00
2022-12-14 13:04:18 +01:00
2022-09-24 14:12:45 +02:00
2021-09-13 10:18:28 -07:00
2022-10-05 11:59:55 +02:00
2022-03-16 12:53:52 +01:00
2022-12-14 13:04:18 +01:00
2022-09-21 17:30:12 +02:00
2022-10-05 11:59:55 +02:00
2022-06-09 15:39:10 +02:00
2022-06-06 11:12:02 +02:00
2021-11-23 11:59:24 +05:30
2022-04-27 16:02:52 +00:00
2022-11-28 16:08:50 +00:00
2022-06-06 11:12:02 +02:00
2022-06-06 11:12:02 +02:00
2022-09-28 13:32:32 +02:00
2021-11-23 11:59:24 +05:30
2022-07-13 19:41:43 +02:00
2022-12-08 16:43:35 +00:00
2022-07-09 11:22:04 +02:00