batman-adv: only add recordroute information to icmp request/reply
Adding host information for record route is only required for ICMP requests and replys, and should not be added to just any (future?) packet type. Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de> Signed-off-by: Marek Lindner <lindner_marek@yahoo.de> Signed-off-by: Antonio Quartulli <antonio@meshcoding.com>
This commit is contained in:
parent
ccdbb6e96b
commit
97dbc03b47
|
@ -379,7 +379,9 @@ int batadv_recv_icmp_packet(struct sk_buff *skb,
|
|||
icmp_packet = (struct batadv_icmp_packet_rr *)skb->data;
|
||||
|
||||
/* add record route information if not full */
|
||||
if ((hdr_size == sizeof(struct batadv_icmp_packet_rr)) &&
|
||||
if ((icmp_packet->msg_type == BATADV_ECHO_REPLY ||
|
||||
icmp_packet->msg_type == BATADV_ECHO_REQUEST) &&
|
||||
(hdr_size == sizeof(struct batadv_icmp_packet_rr)) &&
|
||||
(icmp_packet->rr_cur < BATADV_RR_LEN)) {
|
||||
memcpy(&(icmp_packet->rr[icmp_packet->rr_cur]),
|
||||
ethhdr->h_dest, ETH_ALEN);
|
||||
|
|
Loading…
Reference in New Issue