Skip to content

Commit

Permalink
crypto: af_alg - Add nokey compatibility path
Browse files Browse the repository at this point in the history
This patch adds a compatibility path to support old applications
that do acept(2) before setkey.

Cc: stable@vger.kernel.org
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
  • Loading branch information
Herbert Xu committed Jan 18, 2016
1 parent a383292 commit 3776658
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
13 changes: 12 additions & 1 deletion crypto/af_alg.c
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ int af_alg_register_type(const struct af_alg_type *type)
goto unlock;

type->ops->owner = THIS_MODULE;
if (type->ops_nokey)
type->ops_nokey->owner = THIS_MODULE;
node->type = type;
list_add(&node->list, &alg_types);
err = 0;
Expand Down Expand Up @@ -267,6 +269,7 @@ int af_alg_accept(struct sock *sk, struct socket *newsock)
const struct af_alg_type *type;
struct sock *sk2;
int err;
bool nokey;

lock_sock(sk);
type = ask->type;
Expand All @@ -285,19 +288,27 @@ int af_alg_accept(struct sock *sk, struct socket *newsock)
security_sk_clone(sk, sk2);

err = type->accept(ask->private, sk2);

nokey = err == -ENOKEY;
if (nokey && type->accept_nokey)
err = type->accept_nokey(ask->private, sk2);

if (err)
goto unlock;

sk2->sk_family = PF_ALG;

if (!ask->refcnt++)
if (nokey || !ask->refcnt++)
sock_hold(sk);
alg_sk(sk2)->parent = sk;
alg_sk(sk2)->type = type;

newsock->ops = type->ops;
newsock->state = SS_CONNECTED;

if (nokey)
newsock->ops = type->ops_nokey;

err = 0;

unlock:
Expand Down
2 changes: 2 additions & 0 deletions include/crypto/if_alg.h
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,11 @@ struct af_alg_type {
void (*release)(void *private);
int (*setkey)(void *private, const u8 *key, unsigned int keylen);
int (*accept)(void *private, struct sock *sk);
int (*accept_nokey)(void *private, struct sock *sk);
int (*setauthsize)(void *private, unsigned int authsize);

struct proto_ops *ops;
struct proto_ops *ops_nokey;
struct module *owner;
char name[14];
};
Expand Down

0 comments on commit 3776658

Please sign in to comment.