This seems awfully familiar, but does look necessary. Maybe we've just
been getting lucky with offset happening to initialize to zero, as
fragmentation and reassembly (with Tony's patches) does seem to work
without this.
Acked-by: Alan Ott <a...@signal11.us>
Please send this patch to the net-next mailing list.
On 03/08/2013 01:50 AM, Wolf-Bastian Poettner wrote:
> From: Wolf-Bastian Pöttner <poett...@ibr.cs.tu-bs.de>
>
> offset has to be initialized, otherwise the *first* fragment will
> be discarded and reassembly cannot happen.
> ---
> net/ieee802154/6lowpan.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/net/ieee802154/6lowpan.c b/net/ieee802154/6lowpan.c
> index ed3debe..2db7261 100644
> --- a/net/ieee802154/6lowpan.c
> +++ b/net/ieee802154/6lowpan.c
> @@ -745,7 +745,7 @@ lowpan_process_data(struct sk_buff *skb)
> {
> struct lowpan_fragment *frame;
> /* slen stores the rightmost 8 bits of the 11 bits length */
> - u8 slen, offset;
> + u8 slen, offset = 0;
> u16 len, tag;
> bool found = false;
>
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel
