batman-adv: fix skb->data assignment
authorAntonio Quartulli <ordex@autistici.org>
Tue, 19 Jun 2012 09:26:39 +0000 (09:26 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Jun 2012 21:49:08 +0000 (14:49 -0700)
commit2c995ff892313009e336ecc8ec3411022f5b1c39
tree23ce43701aded064555b1efec78114383201269b
parent48d7d0ad9022b36be9fd8a236fb58ad4c0f3b80c
batman-adv: fix skb->data assignment

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 free'd memory.

This bug was introduced by a73105b8d4c765d9ebfb664d0a66802127d8e4c7
(batman-adv: improved client announcement mechanism)

Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Cc: <stable@vger.kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/batman-adv/routing.c