lkml.org 
[lkml]   [2016]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.12 61/64] crypto: algif_hash - Only export and import on sockets with data
    Date
    From: Herbert Xu <herbert@gondor.apana.org.au>

    3.12-stable review patch. If anyone has any objections, please let me know.

    ===============

    commit 4afa5f9617927453ac04b24b584f6c718dfb4f45 upstream.

    The hash_accept call fails to work on sockets that have not received
    any data. For some algorithm implementations it may cause crashes.

    This patch fixes this by ensuring that we only export and import on
    sockets that have received data.

    Reported-by: Harsh Jain <harshjain.prof@gmail.com>
    Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
    Tested-by: Stephan Mueller <smueller@chronox.de>
    Signed-off-by: Jiri Slaby <jslaby@suse.cz>
    ---
    crypto/algif_hash.c | 12 ++++++++++--
    1 file changed, 10 insertions(+), 2 deletions(-)

    diff --git a/crypto/algif_hash.c b/crypto/algif_hash.c
    index 850246206b12..a68b56a368a8 100644
    --- a/crypto/algif_hash.c
    +++ b/crypto/algif_hash.c
    @@ -192,9 +192,14 @@ static int hash_accept(struct socket *sock, struct socket *newsock, int flags)
    struct sock *sk2;
    struct alg_sock *ask2;
    struct hash_ctx *ctx2;
    + bool more;
    int err;

    - err = crypto_ahash_export(req, state);
    + lock_sock(sk);
    + more = ctx->more;
    + err = more ? crypto_ahash_export(req, state) : 0;
    + release_sock(sk);
    +
    if (err)
    return err;

    @@ -205,7 +210,10 @@ static int hash_accept(struct socket *sock, struct socket *newsock, int flags)
    sk2 = newsock->sk;
    ask2 = alg_sk(sk2);
    ctx2 = ask2->private;
    - ctx2->more = 1;
    + ctx2->more = more;
    +
    + if (!more)
    + return err;

    err = crypto_ahash_import(&ctx2->req, state);
    if (err) {
    --
    2.7.1
    \
     
     \ /
      Last update: 2016-02-11 15:21    [W:4.071 / U:0.432 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site