diff options
| author | Alexander Aring <alex.aring@gmail.com> | 2013-02-05 10:23:43 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-02-06 15:56:17 -0500 |
| commit | fc4e98dbbaa30ca2180df0b2b0d0608045ed3f19 (patch) | |
| tree | b8a555d561dfae5ac96cae94c4a8d9f19f7a75d6 /net/ieee802154 | |
| parent | a07fdceccf9d9f1b87f781e9a87662182e590d70 (diff) | |
| download | linux-3.10-fc4e98dbbaa30ca2180df0b2b0d0608045ed3f19.tar.gz linux-3.10-fc4e98dbbaa30ca2180df0b2b0d0608045ed3f19.tar.bz2 linux-3.10-fc4e98dbbaa30ca2180df0b2b0d0608045ed3f19.zip | |
6lowpan: use stack buffer instead of heap
head buffer is only temporary available in lowpan_header_create.
So it's not necessary to put it on the heap.
Also fixed a comment codestyle issue.
Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ieee802154')
| -rw-r--r-- | net/ieee802154/6lowpan.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/net/ieee802154/6lowpan.c b/net/ieee802154/6lowpan.c index f62c3b963f9..43b95ca6111 100644 --- a/net/ieee802154/6lowpan.c +++ b/net/ieee802154/6lowpan.c @@ -377,17 +377,14 @@ static int lowpan_header_create(struct sk_buff *skb, struct ipv6hdr *hdr; const u8 *saddr = _saddr; const u8 *daddr = _daddr; - u8 *head; + u8 head[100]; struct ieee802154_addr sa, da; + /* TODO: + * if this package isn't ipv6 one, where should it be routed? + */ if (type != ETH_P_IPV6) return 0; - /* TODO: - * if this package isn't ipv6 one, where should it be routed? - */ - head = kzalloc(100, GFP_KERNEL); - if (head == NULL) - return -ENOMEM; hdr = ipv6_hdr(skb); hc06_ptr = head + 2; @@ -561,8 +558,6 @@ static int lowpan_header_create(struct sk_buff *skb, skb_pull(skb, sizeof(struct ipv6hdr)); memcpy(skb_push(skb, hc06_ptr - head), head, hc06_ptr - head); - kfree(head); - lowpan_raw_dump_table(__func__, "raw skb data dump", skb->data, skb->len); |