linux-toradex.git/net/ieee802154, branch v3.7-rc5

net/ieee802154/6lowpan.c: Remove unecessary semicolon

2012-09-18T20:08:19+00:00

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin 
Signed-off-by: David S. Miller

netlink: Rename pid to portid to avoid confusion

2012-09-10T19:30:41+00:00

It is a frequent mistake to confuse the netlink port identifier with a
process identifier.  Try to reduce this confusion by renaming fields
that hold port identifiers portid instead of pid.

I have carefully avoided changing the structures exported to
userspace to avoid changing the userspace API.

I have successfully built an allyesconfig kernel with this change.

Signed-off-by: "Eric W. Biederman" 
Acked-by: Stephen Hemminger 
Signed-off-by: David S. Miller

6lowpan: handle NETDEV_UNREGISTER event

2012-09-02T02:48:02+00:00

Before, it was impossible to remove a wpan device which had lowpan
attached to it.

Signed-off-by: Alan Ott 
Signed-off-by: David S. Miller

6lowpan: Make a copy of skb's delivered to 6lowpan

2012-09-02T02:48:01+00:00

Since lowpan_process_data() modifies the skb (by calling skb_pull()), we
need our own copy so that it doesn't affect the data received by other
protcols (in this case, af_ieee802154).

Signed-off-by: Alan Ott 
Signed-off-by: David S. Miller

6lowpan: Change byte order when storing/accessing to len field

2012-07-17T05:52:02+00:00

Lenght field should be encoded using big endian byte order, such as intend in the specs.
As it is currently written, the len field would not be decoded properly on an implementation using the correct byte ordering. Hence, it could lead to interroperability issues.

Also, I rewrote the code so that iphc0 argument of lowpan_alloc_new_frame could be removed.

Signed-off-by: Tony Cheneau 
Signed-off-by: David S. Miller

6lowpan: Change byte order when storing/accessing u16 tag

2012-07-17T05:51:15+00:00

The tag field should be stored and accessed using big endian byte order (as
intended in the specs). Or else, when displayed with a trafic analyser, such a
Wireshark, the field not properly displayed (e.g. 0x01 00 instead of 0x00 01,
and so on).

Signed-off-by: Tony Cheneau 
Signed-off-by: David S. Miller

6lowpan: Fix null pointer dereference in UDP uncompression function

2012-07-17T05:51:15+00:00

When a UDP packet gets fragmented, a crash will occur at reassembly time.
This is because skb->transport_header is not set during earlier period of fragment reassembly.
As a consequence, call to udp_hdr() return NULL and uh (which is NULL) gets
dereferenced without much test.

Signed-off-by: Tony Cheneau 
Signed-off-by: David S. Miller

6lowpan: rework fragment-deleting routine

2012-07-12T14:54:46+00:00

6lowpan module starts collecting incomming frames and fragments
right after lowpan_module_init() therefor it will be better to
clean unfinished fragments in lowpan_cleanup_module() function
instead of doing it when link goes down.

Changed spinlocks type to prevent deadlock with expired timer event
and removed unused one.

Signed-off-by: Alexander Smirnov 
Signed-off-by: David S. Miller

6lowpan: fix tag variable size

2012-07-12T14:54:45+00:00

Function lowpan_alloc_new_frame() takes u8 tag as an argument. However,
its only caller, lowpan_process_data() passes down a u16. Hence,
the tag value can get corrupted. This prevent 6lowpan fragment reassembly of a
message when the fragment tag value is over 256.

Signed-off-by: Alexander Smirnov 
Cc: Tony Cheneau 
Signed-off-by: David S. Miller

6lowpan: get extra headroom in allocated frame

2012-07-12T14:54:45+00:00

Use netdev_alloc_skb_ip_align() instead of alloc_skb() to get some
extra headroom in case we need to forward this frame in a tunnel or
something else.

Signed-off-by: Alexander Smirnov 
Signed-off-by: David S. Miller