lkml.org 
[lkml]   [2017]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.10 184/268] dccp: fix memory leak during tear-down of unsuccessful connection request
    Date
    From: Hannes Frederic Sowa <hannes@stressinduktion.org>

    commit 72ef9c4125c7b257e3a714d62d778ab46583d6a3 upstream.

    This patch fixes a memory leak, which happens if the connection request
    is not fulfilled between parsing the DCCP options and handling the SYN
    (because e.g. the backlog is full), because we forgot to free the
    list of ack vectors.

    Reported-by: Jianwen Ji <jiji@redhat.com>
    Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Willy Tarreau <w@1wt.eu>
    ---
    net/dccp/ccids/ccid2.c | 1 +
    1 file changed, 1 insertion(+)

    diff --git a/net/dccp/ccids/ccid2.c b/net/dccp/ccids/ccid2.c
    index f053198..5e3a730 100644
    --- a/net/dccp/ccids/ccid2.c
    +++ b/net/dccp/ccids/ccid2.c
    @@ -749,6 +749,7 @@ static void ccid2_hc_tx_exit(struct sock *sk)
    for (i = 0; i < hc->tx_seqbufc; i++)
    kfree(hc->tx_seqbuf[i]);
    hc->tx_seqbufc = 0;
    + dccp_ackvec_parsed_cleanup(&hc->tx_av_chunks);
    }

    static void ccid2_hc_rx_packet_recv(struct sock *sk, struct sk_buff *skb)
    --
    2.8.0.rc2.1.gbe9624a
    \
     
     \ /
      Last update: 2017-06-19 21:31    [W:4.068 / U:0.448 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site