summaryrefslogtreecommitdiffstats
path: root/net/batman-adv
diff options
context:
space:
mode:
authorAntonio Quartulli <ordex@autistici.org>2012-03-16 11:52:31 +0100
committerAntonio Quartulli <ordex@autistici.org>2012-05-13 18:24:37 +0200
commit8710e2613a4819aac44f4aed7e29027ac3eeb683 (patch)
tree52a61a7ad228152e6c62111b6472b51bbecaf90c /net/batman-adv
parentf96a8a0b78548c0ec06b0b4b438db6ee895d67e9 (diff)
downloadop-kernel-dev-8710e2613a4819aac44f4aed7e29027ac3eeb683.zip
op-kernel-dev-8710e2613a4819aac44f4aed7e29027ac3eeb683.tar.gz
batman-adv: avoid skb_linearise() if not needed
Whenever we want to access headers only, we do not need to linearise the whole packet. Instead we can use pskb_may_pull() Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Diffstat (limited to 'net/batman-adv')
-rw-r--r--net/batman-adv/routing.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/net/batman-adv/routing.c b/net/batman-adv/routing.c
index 7ed9d8f..4c6467d 100644
--- a/net/batman-adv/routing.c
+++ b/net/batman-adv/routing.c
@@ -916,8 +916,9 @@ static int check_unicast_ttvn(struct bat_priv *bat_priv,
/* Check whether I have to reroute the packet */
if (seq_before(unicast_packet->ttvn, curr_ttvn) || tt_poss_change) {
- /* Linearize the skb before accessing it */
- if (skb_linearize(skb) < 0)
+ /* check if there is enough data before accessing it */
+ if (pskb_may_pull(skb, sizeof(struct unicast_packet) +
+ ETH_HLEN) < 0)
return 0;
ethhdr = (struct ethhdr *)(skb->data +
OpenPOWER on IntegriCloud