Re: [net-next] net: crypto set sk to NULL when af_alg_release.

2019-02-17 Thread maowenan
On 2019/2/18 7:42, David Miller wrote: > From: Mao Wenan > Date: Fri, 15 Feb 2019 22:24:15 +0800 > >> KASAN has found use-after-free in sockfs_setattr. >> The existed commit 6d8c50dcb029 ("socket: close race condition between >> sock_close() >> and sockfs_setattr()") is to fix this simillar i

Re: [net-next] net: crypto set sk to NULL when af_alg_release.

2019-02-17 Thread David Miller
From: Mao Wenan Date: Fri, 15 Feb 2019 22:24:15 +0800 > KASAN has found use-after-free in sockfs_setattr. > The existed commit 6d8c50dcb029 ("socket: close race condition between > sock_close() > and sockfs_setattr()") is to fix this simillar issue, but it seems to ignore > that crypto module fo

Re: [net-next] net: crypto set sk to NULL when af_alg_release.

2019-02-16 Thread maowenan
who can help review this patch? thank you. On 2019/2/15 22:24, Mao Wenan wrote: > KASAN has found use-after-free in sockfs_setattr. > The existed commit 6d8c50dcb029 ("socket: close race condition between > sock_close() > and sockfs_setattr()") is to fix this simillar issue, but it seems to ignor

[net-next] net: crypto set sk to NULL when af_alg_release.

2019-02-15 Thread Mao Wenan
KASAN has found use-after-free in sockfs_setattr. The existed commit 6d8c50dcb029 ("socket: close race condition between sock_close() and sockfs_setattr()") is to fix this simillar issue, but it seems to ignore that crypto module forgets to set the sk to NULL after af_alg_release. KASAN report de