lkml.org 
[lkml]   [2012]   [Aug]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[ 029/109] batman-adv: fix skb->data assignment
    Date
    From: Greg KH <gregkh@linuxfoundation.org>

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

    ------------------

    From: Antonio Quartulli <ordex@autistici.org>

    commit d2b6cc8e460494251442a877fcbc150faa175b4f upstream.

    skb_linearize(skb) possibly rearranges the skb internal data and then changes
    the skb->data pointer value. For this reason any other pointer in the code that
    was assigned skb->data before invoking skb_linearise(skb) must be re-assigned.

    In the current tt_query message handling code this is not done and therefore, in
    case of skb linearization, the pointer used to handle the packet header ends up
    in pointing to poisoned memory. The packet is then dropped but the
    translation-table mechanism is corrupted.

    Signed-off-by: Antonio Quartulli <ordex@autistici.org>
    Signed-off-by: Sven Eckelmann <sven@narfation.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    net/batman-adv/routing.c | 2 ++
    1 file changed, 2 insertions(+)

    --- a/net/batman-adv/routing.c
    +++ b/net/batman-adv/routing.c
    @@ -627,6 +627,8 @@ int recv_tt_query(struct sk_buff *skb, s
    if (unlikely(skb_headlen(skb) <
    sizeof(struct tt_query_packet) + tt_len))
    goto out;
    + /* skb_linearize() possibly changed skb->data */
    + tt_query = (struct tt_query_packet *)skb->data;

    handle_tt_response(bat_priv, tt_query);
    } else {



    \
     
     \ /
      Last update: 2012-08-08 02:01    [W:3.362 / U:0.832 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site