diff options
| author |   Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2019-09-20 21:45:33 +0200 |
|---|---|---|
| committer |   Stefan Schmidt <stefan@datenfreihafen.org> | 2019-09-25 08:16:51 +0200 |
| commit | 21a045e430e56725628f361dede8a882e92469b9 (patch) | |
| tree | 838d1625abfaef6108a224d2b96844fbb0f17807 /drivers/net/ieee802154 | |
| parent | ieee802154: atusb: fix use-after-free at disconnect (diff) | |
| download | linux-dev-21a045e430e56725628f361dede8a882e92469b9.tar.xz linux-dev-21a045e430e56725628f361dede8a882e92469b9.zip | |
ieee802154: mcr20a: simplify a bit 'mcr20a_handle_rx_read_buf_complete()'
Use a 'skb_put_data()' variant instead of rewritting it.
The __skb_put_data variant is safe here. It is obvious that the skb can
not overflow. It has just been allocated a few lines above with the same
'len'.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Acked-by: Xue Liu <liuxuenetmail@gmail.com>
Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org>
Diffstat (limited to 'drivers/net/ieee802154')
| -rw-r--r-- | drivers/net/ieee802154/mcr20a.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ieee802154/mcr20a.c b/drivers/net/ieee802154/mcr20a.c index 17f2300e63ee..8dc04e2590b1 100644 --- a/drivers/net/ieee802154/mcr20a.c +++ b/drivers/net/ieee802154/mcr20a.c @@ -800,7 +800,7 @@ mcr20a_handle_rx_read_buf_complete(void *context) if (!skb) return; - memcpy(skb_put(skb, len), lp->rx_buf, len); + __skb_put_data(skb, lp->rx_buf, len); ieee802154_rx_irqsafe(lp->hw, skb, lp->rx_lqi[0]); print_hex_dump_debug("mcr20a rx: ", DUMP_PREFIX_OFFSET, 16, 1, |