[B.A.T.M.A.N.] [PATCH-next] batman-adv: fix batman-adv header overhead calculation

[B.A.T.M.A.N.] [PATCH-next] batman-adv: fix batman-adv header overhead calculation

[Marek Lindner]

Batman-adv prepends a full ethernet header in addition to its own
header. This has to be reflected in the MTU calculation.

Introduced by f1f310d5c261f0214f85afea95d5cd11d49d64c6
("batman-adv: consider network coding overhead when calculating required mtu")

Reported-by: cmsv <cmsv@wirelesspt.net>
Reported-by: Martin Hundebøll <martin@hundeboll.net>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
---
 main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/main.c b/main.c
index 4547bf0..fd85205 100644
--- a/main.c
+++ b/main.c
@@ -277,7 +277,7 @@ int batadv_max_header_len(void)
 			   sizeof(struct batadv_coded_packet));
 #endif
 
-	return header_len;
+	return header_len + ETH_HLEN;
 }
 
 /**
-- 
1.8.4.4

Re: [B.A.T.M.A.N.] [PATCH-next] batman-adv: fix batman-adv header overhead calculation

[Marek Lindner]

[-- Attachment #1: Type: text/plain, Size: 626 bytes --]

On Wednesday 15 January 2014 20:39:33 Marek Lindner wrote:
> Batman-adv prepends a full ethernet header in addition to its own
> header. This has to be reflected in the MTU calculation.
> 
> Introduced by f1f310d5c261f0214f85afea95d5cd11d49d64c6
> ("batman-adv: consider network coding overhead when calculating required
> mtu")
> 
> Reported-by: cmsv <cmsv@wirelesspt.net>
> Reported-by: Martin Hundebøll <martin@hundeboll.net>
> Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
> ---
>  main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied in revision 746d643.

Regards,
Marek

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 490 bytes --]