[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
URL:
  <http://savannah.nongnu.org/bugs/?48825>

                 Summary: lowpan6: source PAN ID is skipped without checking
PAN ID compression flag
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: authmillenon
            Submitted on: Fri 19 Aug 2016 11:02:51 AM GMT
                Category: None
                Severity: 3 - Normal
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None
            lwIP version: git head

    _______________________________________________________

Details:

Hi,

In lowpan6_input() (src/netif/lowpan6.c) the source PAN ID in the IEEE
802.15.4 header is skipped (in current version it is even commented with "/*
skip PAN ID. */". However, according to the IEEE 802.15.4 specs this field can
also be omitted, if the PAN_COMP bit is set the FCF. So if data comes in from
another implementation (e.g. Contiki's uIP or RIOT's GNRC) the packet is read
incorrectly (basically anything that would come after the source PAN), if they
have PAN compression activated.

Kind Regards,
Martine Lenders




    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #1, bug #48825 (project lwip):

We have at least 2 bugs open for the 6lowpan netif (which is still *very*
experimental). Does this mean the rest works (e.g. against RIOT or uIP)?

Unfortunately, I have no working counterpart to test 6lowpan. Is there some
kind of debug environment available where I can simulate a 6lowpan environment
e.g. on a PC?

BTW/OT: congrats to your c't article ;-)

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #2, bug #48825 (project lwip):

Is anyone even reading this? ;-)

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #3, bug #48825 (project lwip):

Yes, sorry for not replying! I was a little bit swamped in other work.

> We have at least 2 bugs open for the 6lowpan netif (which is still very
experimental). Does this mean the rest works (e.g. against RIOT or uIP)?

That's the main reason I did not reply, because at the moment I did not find
the time to confirm this. Last time I tried I basically hit the bug reported
here and did not bother to test further. Let me come back to you in a few days
about that.

> Unfortunately, I have no working counterpart to test 6lowpan. Is there some
kind of debug environment available where I can simulate a 6lowpan environment
e.g. on a PC?

You could use RIOT's native port for that e.g. I did not test it yet (and you
need a very up-to-date Linux kernel for this), but Alexander Aring who did the
6Lo port for Linux provided a simulated 802.15.4 device for RIOT [1].

Alternatively, testing and debugging on the FIT/IoT-LAB testbed [2] works
quite smoothly (you need to register an account though, but for a member of a
FOSS project this shouldn't be a problem to my experience).

> BTW/OT: congrats to your c't article ;-)

Thanks! Though I had nothing to do with that I will carry that to the people
who do :-).

[1] https://github.com/RIOT-OS/RIOT/pull/5582
[2] https://www.iot-lab.info/

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Update of bug #48825 (project lwip):

              Item Group:                    None => Faulty Behaviour      
                  Status:                    None => Fixed                  
             Assigned to:                    None => goldsimon              
             Open/Closed:                    Open => Closed                

    _______________________________________________________

Follow-up Comment #4:

I did a round of spec-reading and coding, so this should be fixed.
Unfortunately, I don't have hardware to test 6LoWPAN, so I hope I did not
break things while trying to fix this.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #5, bug #48825 (project lwip):

Sorry again for the radio silence :-/. I can't test either right now (mostly
because some porting effort would be required to test the current commit in my
set-up), but the changes look sane. Maybe for our next release tests (mid- to
end of April) we can include this again.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #6, bug #48825 (project lwip):

No problem. I can't test it either. I'm working on tunneling it in ZigBee
Encapsulation Protocol. That way, wireshark shows me the contents and maybe
I'll get a working remove station...

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel
Reply | Threaded
Open this post in threaded view
|

[bug #48825] lowpan6: source PAN ID is skipped without checking PAN ID compression flag

Simon Goldschmidt
Follow-up Comment #7, bug #48825 (project lwip):

FYI: I tested for https://github.com/RIOT-OS/RIOT/pull/10059 the interaction
with alternative stacks and it worked. So now I can confirm that this issue is
fixed.

    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?48825>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/


_______________________________________________
lwip-devel mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/lwip-devel