[maint] batman-adv: remove useless assignment

Message ID 1391086647-1732-1-git-send-email-antonio@meshcoding.com (mailing list archive)
State Accepted, archived
Commit 9a91d33c0ea234faf59ee7543d5253e6809a6b6f
Headers

Commit Message

Antonio Quartulli Jan. 30, 2014, 12:57 p.m. UTC
  Introduced by 41b38727749a94c1a65cf0f4be9bfe1cbaf0adeb
   ("batman-adv: fix potential kernel paging error for unicast transmissions")

Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
---
 send.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Marek Lindner Feb. 2, 2014, 2:43 p.m. UTC | #1
On Thursday 30 January 2014 13:57:27 Antonio Quartulli wrote:
> Introduced by 41b38727749a94c1a65cf0f4be9bfe1cbaf0adeb
>    ("batman-adv: fix potential kernel paging error for unicast
> transmissions")
> 
> Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
> ---
>  send.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied in revision 9a91d33.

Thanks,
Marek
  

Patch

diff --git a/send.c b/send.c
index 2035bd9..75be770 100644
--- a/send.c
+++ b/send.c
@@ -256,7 +256,7 @@  static int batadv_send_skb_unicast(struct batadv_priv *bat_priv,
 				   struct batadv_orig_node *orig_node,
 				   unsigned short vid)
 {
-	struct ethhdr *ethhdr = (struct ethhdr *)skb->data;
+	struct ethhdr *ethhdr;
 	struct batadv_unicast_packet *unicast_packet;
 	int ret = NET_XMIT_DROP;