]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
crypto: af_alg - Disallow bind/setkey/... after accept(2)
authorHerbert Xu <herbert@gondor.apana.org.au>
Wed, 30 Dec 2015 03:47:53 +0000 (11:47 +0800)
committerSasha Levin <sasha.levin@oracle.com>
Wed, 3 Feb 2016 21:23:18 +0000 (16:23 -0500)
commit0571ba52a19e18a1c20469454231eef681cb1310
tree73e9241c6d91b22bc5d0e796b7c4a647aa9834fa
parentbc24ac15b0746172a8f603171352aa54abcf7c78
crypto: af_alg - Disallow bind/setkey/... after accept(2)

[ Upstream commit c840ac6af3f8713a71b4d2363419145760bd6044 ]

Each af_alg parent socket obtained by socket(2) corresponds to a
tfm object once bind(2) has succeeded.  An accept(2) call on that
parent socket creates a context which then uses the tfm object.

Therefore as long as any child sockets created by accept(2) exist
the parent socket must not be modified or freed.

This patch guarantees this by using locks and a reference count
on the parent socket.  Any attempt to modify the parent socket will
fail with EBUSY.

Cc: stable@vger.kernel.org
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
crypto/af_alg.c
include/crypto/if_alg.h