• K
    oom: __task_cred() need rcu_read_lock() · 8d6c83f0
    KOSAKI Motohiro 提交于
    dump_tasks() needs to hold the RCU read lock around its access of the
    target task's UID.  To this end it should use task_uid() as it only needs
    that one thing from the creds.
    
    The fact that dump_tasks() holds tasklist_lock is insufficient to prevent the
    target process replacing its credentials on another CPU.
    
    Then, this patch change to call rcu_read_lock() explicitly.
    
    	===================================================
    	[ INFO: suspicious rcu_dereference_check() usage. ]
    	---------------------------------------------------
    	mm/oom_kill.c:410 invoked rcu_dereference_check() without protection!
    
    	other info that might help us debug this:
    
    	rcu_scheduler_active = 1, debug_locks = 1
    	4 locks held by kworker/1:2/651:
    	 #0:  (events){+.+.+.}, at: [<ffffffff8106aae7>]
    	process_one_work+0x137/0x4a0
    	 #1:  (moom_work){+.+...}, at: [<ffffffff8106aae7>]
    	process_one_work+0x137/0x4a0
    	 #2:  (tasklist_lock){.+.+..}, at: [<ffffffff810fafd4>]
    	out_of_memory+0x164/0x3f0
    	 #3:  (&(&p->alloc_lock)->rlock){+.+...}, at: [<ffffffff810fa48e>]
    	find_lock_task_mm+0x2e/0x70
    Signed-off-by: NKOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
    Signed-off-by: NDavid Howells <dhowells@redhat.com>
    Acked-by: NPaul E. McKenney <paulmck@linux.vnet.ibm.com>
    Acked-by: NDavid Rientjes <rientjes@google.com>
    Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
    8d6c83f0
oom_kill.c 19.9 KB