Hello,

I'm reading the 6lowpan.c file and I find the following code very odd. 
I believe a bug lies in it. I cannot test at the moment because the 
serial driver I plan on using does not work with the 6lowpan stack.

In lowpan_header_create(), there is the following switch (around line 
495):
>       switch (hdr->hop_limit) {
>       case 1:
>               iphc0 |= LOWPAN_IPHC_TTL_1;
>               break;
>       case 64:
>               iphc0 |= LOWPAN_IPHC_TTL_64;
>               break;
>       case 255:
>               iphc0 |= LOWPAN_IPHC_TTL_255;
>               break;
>       default:
>               *hc06_ptr = hdr->hop_limit;
>               break;
>       }

I believe the code in the default block should be:
>       default:
>               *hc06_ptr = hdr->hop_limit;
>               hc06_ptr += 1;
>               break;
>       }

Can you confirm this is indeed a bug?

Regards,
Tony


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to