lkml.org 
[lkml]   [2007]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 07/59] NETFILTER: nf_conntrack_ipv6: fix crash when handling fragments
    -stable review patch.  If anyone has any objections, please let us know.
    ------------------

    From: Patrick McHardy <kaber@trash.net>

    When IPv6 connection tracking splits up a defragmented packet into
    its original fragments, the packets are taken from a list and are
    passed to the network stack with skb->next still set. This causes
    dev_hard_start_xmit to treat them as GSO fragments, resulting in
    a use after free when connection tracking handles the next fragment.

    Signed-off-by: Patrick McHardy <kaber@trash.net>
    Signed-off-by: Chris Wright <chrisw@sous-sol.org>
    ---

    net/ipv6/netfilter/nf_conntrack_reasm.c | 2 ++
    1 file changed, 2 insertions(+)

    --- linux-2.6.19.2.orig/net/ipv6/netfilter/nf_conntrack_reasm.c
    +++ linux-2.6.19.2/net/ipv6/netfilter/nf_conntrack_reasm.c
    @@ -835,6 +835,8 @@ void nf_ct_frag6_output(unsigned int hoo
    s->nfct_reasm = skb;

    s2 = s->next;
    + s->next = NULL;
    +
    NF_HOOK_THRESH(PF_INET6, hooknum, s, in, out, okfn,
    NF_IP6_PRI_CONNTRACK_DEFRAG + 1);
    s = s2;
    --
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-02-03 03:57    [W:0.032 / U:61.708 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site