net: manual clean code which call skb_put_[data:zero]
authoryuan linyu <Linyu.Yuan@alcatel-sbell.com.cn>
Sun, 18 Jun 2017 14:52:04 +0000 (22:52 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 20 Jun 2017 17:30:15 +0000 (13:30 -0400)
Signed-off-by: yuan linyu <Linyu.Yuan@alcatel-sbell.com.cn>
Signed-off-by: David S. Miller <davem@davemloft.net>
39 files changed:
drivers/isdn/hysdn/hycapi.c
drivers/isdn/i4l/isdn_bsdcomp.c
drivers/isdn/i4l/isdn_ppp.c
drivers/net/bonding/bond_alb.c
drivers/net/caif/caif_hsi.c
drivers/net/caif/caif_serial.c
drivers/net/caif/caif_spi.c
drivers/net/ethernet/nxp/lpc_eth.c
drivers/net/ethernet/packetengines/hamachi.c
drivers/net/ppp/ppp_synctty.c
drivers/net/usb/asix_common.c
drivers/net/usb/hso.c
drivers/net/wireless/ath/ath9k/wmi.c
drivers/net/wireless/marvell/libertas/if_sdio.c
drivers/net/wireless/quantenna/qtnfmac/qlink_util.h
drivers/net/wireless/realtek/rtlwifi/pci.c
drivers/net/wireless/rsi/rsi_91x_mgmt.c
drivers/net/wireless/ti/wlcore/rx.c
drivers/nfc/pn533/pn533.c
drivers/staging/rtl8188eu/core/rtw_recv.c
drivers/staging/rtl8192e/rtllib_rx.c
drivers/staging/rtl8192e/rtllib_softmac.c
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
drivers/staging/rtl8192u/r819xU_cmdpkt.c
drivers/staging/rtl8712/rtl8712_recv.c
drivers/staging/rtl8723bs/os_dep/recv_linux.c
drivers/staging/wlan-ng/hfa384x_usb.c
drivers/usb/gadget/function/f_ncm.c
lib/nlattr.c
net/bridge/netfilter/nft_reject_bridge.c
net/dsa/tag_trailer.c
net/hsr/hsr_device.c
net/irda/irlap_frame.c
net/mac80211/mlme.c
net/mac80211/tdls.c
net/sctp/sm_make_chunk.c
net/vmw_vsock/virtio_transport_common.c
net/x25/x25_subr.c

index 87119b51750894f0db79e2447d4f1789a56195fd..eac0f51a0f607c5e40b0c5e06d47b5527fb1d30d 100644 (file)
@@ -173,8 +173,8 @@ hycapi_register_internal(struct capi_ctr *ctrl, __u16 appl,
        }
        skb_put_data(skb, &len, sizeof(__u16));
        skb_put_data(skb, &appl, sizeof(__u16));
-       memcpy(skb_put(skb, sizeof(__u8)), &_command, sizeof(_command));
-       memcpy(skb_put(skb, sizeof(__u8)), &_subcommand, sizeof(_subcommand));
+       skb_put_data(skb, &_command, sizeof(__u8));
+       skb_put_data(skb, &_subcommand, sizeof(__u8));
        skb_put_data(skb, &MessageNumber, sizeof(__u16));
        skb_put_data(skb, &MessageBufferSize, sizeof(__u16));
        skb_put_data(skb, &(rp->level3cnt), sizeof(__u16));
@@ -281,8 +281,8 @@ static void hycapi_release_internal(struct capi_ctr *ctrl, __u16 appl)
        }
        skb_put_data(skb, &len, sizeof(__u16));
        skb_put_data(skb, &appl, sizeof(__u16));
-       memcpy(skb_put(skb, sizeof(__u8)), &_command, sizeof(_command));
-       memcpy(skb_put(skb, sizeof(__u8)), &_subcommand, sizeof(_subcommand));
+       skb_put_data(skb, &_command, sizeof(__u8));
+       skb_put_data(skb, &_subcommand, sizeof(__u8));
        skb_put_data(skb, &MessageNumber, sizeof(__u16));
        hycapi_send_message(ctrl, skb);
        hycapi_applications[appl - 1].ctrl_mask &= ~(1 << (ctrl->cnr - 1));
index 3035210a6119edc8fa4bdf05605d9d6a09868fe9..5b64a1389a7ca6035e20b0018459055fdc63999b 100644 (file)
@@ -472,7 +472,7 @@ static int bsd_compress(void *state, struct sk_buff *skb_in, struct sk_buff *skb
                accm |= ((ent) << bitno);                               \
                do      {                                               \
                        if (skb_out && skb_tailroom(skb_out) > 0)       \
-                               *(u8 *)skb_put(skb_out, 1) = (u8)(accm >> 24); \
+                               skb_put(skb_out, (u8)(accm >> 24));     \
                        accm <<= 8;                                     \
                        bitno += 8;                                     \
                } while (bitno <= 24);                                  \
index b7e3f1cde683abcfb8d884853bf3204b48973650..88e5a025cea7163bf7b5a59b0fccd587fd423bd4 100644 (file)
@@ -2258,7 +2258,7 @@ static void isdn_ppp_ccp_xmit_reset(struct ippp_struct *is, int proto,
 
        /* Now stuff remaining bytes */
        if (len) {
-               p = skb_put_data(skb, data, len);
+               skb_put_data(skb, data, len);
        }
 
        /* skb is now ready for xmit */
index b796db7dd621f9e3612e08b33e132cc94a8eaf15..c02cc817a490995498b9764594a632b7581a700c 100644 (file)
@@ -925,7 +925,6 @@ static void alb_send_lp_vid(struct slave *slave, u8 mac_addr[],
        struct learning_pkt pkt;
        struct sk_buff *skb;
        int size = sizeof(struct learning_pkt);
-       char *data;
 
        memset(&pkt, 0, size);
        ether_addr_copy(pkt.mac_dst, mac_addr);
@@ -936,7 +935,7 @@ static void alb_send_lp_vid(struct slave *slave, u8 mac_addr[],
        if (!skb)
                return;
 
-       data = skb_put_data(skb, &pkt, size);
+       skb_put_data(skb, &pkt, size);
 
        skb_reset_mac_header(skb);
        skb->network_header = skb->mac_header + ETH_HLEN;
index 4534326e20ac7bb2e752472cf413e4fd64d3d5e6..11ba6e3eea2284081133d714bf5dcfb87ddf9b8d 100644 (file)
@@ -426,7 +426,6 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
        /* Check for embedded CAIF frame. */
        if (desc->offset) {
                struct sk_buff *skb;
-               u8 *dst = NULL;
                int len = 0;
                pfrm = ((u8 *)desc) + desc->offset;
 
@@ -454,7 +453,7 @@ static int cfhsi_rx_desc(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                }
                caif_assert(skb != NULL);
 
-               dst = skb_put_data(skb, pfrm, len);
+               skb_put_data(skb, pfrm, len);
 
                skb->protocol = htons(ETH_P_CAIF);
                skb_reset_mac_header(skb);
@@ -555,7 +554,6 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
        /* Parse payload. */
        while (nfrms < CFHSI_MAX_PKTS && *plen) {
                struct sk_buff *skb;
-               u8 *dst = NULL;
                u8 *pcffrm = NULL;
                int len;
 
@@ -584,7 +582,7 @@ static int cfhsi_rx_pld(struct cfhsi_desc *desc, struct cfhsi *cfhsi)
                }
                caif_assert(skb != NULL);
 
-               dst = skb_put_data(skb, pcffrm, len);
+               skb_put_data(skb, pcffrm, len);
 
                skb->protocol = htons(ETH_P_CAIF);
                skb_reset_mac_header(skb);
index 5c57be2082bacf484d1621c7bc4aac09a3fb1bf2..709838e4c06228c68094bdef40f934f4696f9871 100644 (file)
@@ -171,7 +171,6 @@ static void ldisc_receive(struct tty_struct *tty, const u8 *data,
        struct sk_buff *skb = NULL;
        struct ser_device *ser;
        int ret;
-       u8 *p;
 
        ser = tty->disc_data;
 
@@ -198,7 +197,7 @@ static void ldisc_receive(struct tty_struct *tty, const u8 *data,
        skb = netdev_alloc_skb(ser->dev, count+1);
        if (skb == NULL)
                return;
-       p = skb_put_data(skb, data, count);
+       skb_put_data(skb, data, count);
 
        skb->protocol = htons(ETH_P_CAIF);
        skb_reset_mac_header(skb);
index 24a5f5ca2037d91c05ea36782adeb63315726cb5..207cb8423de0f315be2b3fcf92b52d41784986fe 100644 (file)
@@ -526,7 +526,6 @@ int cfspi_rxfrm(struct cfspi *cfspi, u8 *buf, size_t len)
                struct sk_buff *skb = NULL;
                int spad = 0;
                int epad = 0;
-               u8 *dst = NULL;
                int pkt_len = 0;
 
                /*
@@ -548,7 +547,7 @@ int cfspi_rxfrm(struct cfspi *cfspi, u8 *buf, size_t len)
                skb = netdev_alloc_skb(cfspi->ndev, pkt_len + 1);
                caif_assert(skb != NULL);
 
-               dst = skb_put_data(skb, src, pkt_len);
+               skb_put_data(skb, src, pkt_len);
                src += pkt_len;
 
                skb->protocol = htons(ETH_P_CAIF);
index 828bfd93cb54a5ab200d2e47913899a99475679e..08381ef8bdb481df81eb448b2f8ceda61c13ba8e 100644 (file)
@@ -919,7 +919,6 @@ static int __lpc_handle_recv(struct net_device *ndev, int budget)
        struct sk_buff *skb;
        u32 rxconsidx, len, ethst;
        struct rx_status_t *prxstat;
-       u8 *prdbuf;
        int rx_done = 0;
 
        /* Get the current RX buffer indexes */
@@ -960,9 +959,9 @@ static int __lpc_handle_recv(struct net_device *ndev, int budget)
                                ndev->stats.rx_dropped++;
                        } else {
                                /* Copy packet from buffer */
-                               prdbuf = skb_put_data(skb,
-                                                     pldat->rx_buff_v + rxconsidx * ENET_MAXF_SIZE,
-                                                     len);
+                               skb_put_data(skb,
+                                            pldat->rx_buff_v + rxconsidx * ENET_MAXF_SIZE,
+                                            len);
 
                                /* Pass to upper layer */
                                skb->protocol = eth_type_trans(skb, ndev);
index 8b026dbf0d8dccbbeee916c4cc94e8d26e43c944..482b85e4d66506c21ed992932887aa007f357b82 100644 (file)
@@ -1495,8 +1495,8 @@ static int hamachi_rx(struct net_device *dev)
                                        hmp->rx_skbuff[entry]->data, pkt_len);
                                skb_put(skb, pkt_len);
 #else
-                               memcpy(skb_put(skb, pkt_len), hmp->rx_ring_dma
-                                       + entry*sizeof(*desc), pkt_len);
+                               skb_put_data(skb, hmp->rx_ring_dma
+                                            + entry*sizeof(*desc), pkt_len);
 #endif
                                pci_dma_sync_single_for_device(hmp->pci_dev,
                                                               leXX_to_cpu(hmp->rx_ring[entry].addr),
index ef08590db873203691963fd67e8341e23a5c3d23..7868c29071d4fa3dd63f51bf7ecd415ac9e18181 100644 (file)
@@ -697,7 +697,7 @@ ppp_sync_input(struct syncppp *ap, const unsigned char *buf,
                goto err;
        }
 
-       p = skb_put_data(skb, buf, count);
+       skb_put_data(skb, buf, count);
 
        /* strip address/control field if present */
        p = skb->data;
index 90facc5ecab0e0db356c175419c21fe87e26a3d9..7847436c441e3c16b91fb0d72c3f07061d4e772d 100644 (file)
@@ -113,7 +113,6 @@ int asix_rx_fixup_internal(struct usbnet *dev, struct sk_buff *skb,
 
        while (offset + sizeof(u16) <= skb->len) {
                u16 copy_length;
-               unsigned char *data;
 
                if (!rx->remaining) {
                        if (skb->len - offset == sizeof(u16)) {
@@ -167,8 +166,8 @@ int asix_rx_fixup_internal(struct usbnet *dev, struct sk_buff *skb,
                }
 
                if (rx->ax_skb) {
-                       data = skb_put_data(rx->ax_skb, skb->data + offset,
-                                           copy_length);
+                       skb_put_data(rx->ax_skb, skb->data + offset,
+                                    copy_length);
                        if (!rx->remaining)
                                usbnet_skb_return(dev, rx->ax_skb);
                }
index 908ada4ca21c97ac96748c16bd77b5eb2b48797e..d7a3379ea668f7b8e35102e080e41931ae1d1049 100644 (file)
@@ -861,7 +861,6 @@ static void packetizeRx(struct hso_net *odev, unsigned char *ip_pkt,
        unsigned short temp_bytes;
        unsigned short buffer_offset = 0;
        unsigned short frame_len;
-       unsigned char *tmp_rx_buf;
 
        /* log if needed */
        hso_dbg(0x1, "Rx %d bytes\n", count);
@@ -911,9 +910,9 @@ static void packetizeRx(struct hso_net *odev, unsigned char *ip_pkt,
 
                                /* Copy what we got so far. make room for iphdr
                                 * after tail. */
-                               tmp_rx_buf = skb_put_data(odev->skb_rx_buf,
-                                                         (char *)&(odev->rx_ip_hdr),
-                                                         sizeof(struct iphdr));
+                               skb_put_data(odev->skb_rx_buf,
+                                            (char *)&(odev->rx_ip_hdr),
+                                            sizeof(struct iphdr));
 
                                /* ETH_HLEN */
                                odev->rx_buf_size = sizeof(struct iphdr);
@@ -932,9 +931,9 @@ static void packetizeRx(struct hso_net *odev, unsigned char *ip_pkt,
                        /* Copy the rest of the bytes that are left in the
                         * buffer into the waiting sk_buf. */
                        /* Make room for temp_bytes after tail. */
-                       tmp_rx_buf = skb_put_data(odev->skb_rx_buf,
-                                                 ip_pkt + buffer_offset,
-                                                 temp_bytes);
+                       skb_put_data(odev->skb_rx_buf,
+                                    ip_pkt + buffer_offset,
+                                    temp_bytes);
 
                        odev->rx_buf_missing -= temp_bytes;
                        count -= temp_bytes;
index 85d09fdef8dc18a872a5b61c5abdebe6267e1c4f..64a354fa78ab2f79b7811e7119886f7c463473cc 100644 (file)
@@ -298,7 +298,6 @@ int ath9k_wmi_cmd(struct wmi *wmi, enum wmi_cmd_id cmd_id,
        u16 headroom = sizeof(struct htc_frame_hdr) +
                       sizeof(struct wmi_cmd_hdr);
        struct sk_buff *skb;
-       u8 *data;
        unsigned long time_left;
        int ret = 0;
 
@@ -312,7 +311,7 @@ int ath9k_wmi_cmd(struct wmi *wmi, enum wmi_cmd_id cmd_id,
        skb_reserve(skb, headroom);
 
        if (cmd_len != 0 && cmd_buf != NULL) {
-               data = skb_put_data(skb, cmd_buf, cmd_len);
+               skb_put_data(skb, cmd_buf, cmd_len);
        }
 
        mutex_lock(&wmi->op_mutex);
index a9e2b06b3175c299a0757775a121cb13caabfdb5..2300e796c6ab9e8106e132b3988fa240dc4783a4 100644 (file)
@@ -239,7 +239,6 @@ static int if_sdio_handle_data(struct if_sdio_card *card,
 {
        int ret;
        struct sk_buff *skb;
-       char *data;
 
        if (size > MRVDRV_ETH_RX_PACKET_BUFFER_SIZE) {
                lbs_deb_sdio("response packet too large (%d bytes)\n",
@@ -256,7 +255,7 @@ static int if_sdio_handle_data(struct if_sdio_card *card,
 
        skb_reserve(skb, NET_IP_ALIGN);
 
-       data = skb_put_data(skb, buffer, size);
+       skb_put_data(skb, buffer, size);
 
        lbs_process_rxed_packet(card->priv, skb);
 
index f6ac39973b5d6419432f9fa5af1df1e1a2bd3cc3..90d7d09a6c6329493482a5fb0cc70835edadd602 100644 (file)
@@ -33,9 +33,7 @@ static inline void qtnf_cmd_skb_put_action(struct sk_buff *skb, u16 action)
 static inline void
 qtnf_cmd_skb_put_buffer(struct sk_buff *skb, const u8 *buf_src, size_t len)
 {
-       u8 *buf_dst;
-
-       buf_dst = skb_put_data(skb, buf_src, len);
+       skb_put_data(skb, buf_src, len);
 }
 
 static inline void qtnf_cmd_skb_put_tlv_arr(struct sk_buff *skb,
index 0c1f8307e1796bc32d7c0b008298b2453c395881..df5f6795f65090dafa6df8c6bdcc666c81d9af33 100644 (file)
@@ -729,13 +729,12 @@ static void _rtl_pci_rx_to_mac80211(struct ieee80211_hw *hw,
                dev_kfree_skb_any(skb);
        } else {
                struct sk_buff *uskb = NULL;
-               u8 *pdata;
 
                uskb = dev_alloc_skb(skb->len + 128);
                if (likely(uskb)) {
                        memcpy(IEEE80211_SKB_RXCB(uskb), &rx_status,
                               sizeof(rx_status));
-                       pdata = skb_put_data(uskb, skb->data, skb->len);
+                       skb_put_data(uskb, skb->data, skb->len);
                        dev_kfree_skb_any(skb);
                        ieee80211_rx_irqsafe(hw, uskb);
                } else {
index 4433cec4367ccc20c80bda236e8171da449d6926..a0f04371d93b906250f52c945fd0be07fd38f5a4 100644 (file)
@@ -389,9 +389,7 @@ static int rsi_mgmt_pkt_to_core(struct rsi_common *common,
        struct ieee80211_tx_info *info;
        struct skb_info *rx_params;
        u8 pad_bytes = msg[4];
-       u8 pkt_recv;
        struct sk_buff *skb;
-       char *buffer;
 
        if (type == RX_DOT11_MGMT) {
                if (!adapter->sc_nvifs)
@@ -412,11 +410,9 @@ static int rsi_mgmt_pkt_to_core(struct rsi_common *common,
                        return -ENOMEM;
                }
 
-               buffer = skb_put_data(skb,
-                                     (u8 *)(msg + FRAME_DESC_SZ + pad_bytes),
-                                     msg_len);
-
-               pkt_recv = buffer[0];
+               skb_put_data(skb,
+                            (u8 *)(msg + FRAME_DESC_SZ + pad_bytes),
+                            msg_len);
 
                info = IEEE80211_SKB_CB(skb);
                rx_params = (struct skb_info *)info->driver_data;
index 53cd6d4d5b505d644553c27e45c5fd9bb1217f44..0f15696195f884f59c82e431757593362a0489ca 100644 (file)
@@ -117,7 +117,6 @@ static int wl1271_rx_handle_data(struct wl1271 *wl, u8 *data, u32 length,
        struct wl1271_rx_descriptor *desc;
        struct sk_buff *skb;
        struct ieee80211_hdr *hdr;
-       u8 *buf;
        u8 beacon = 0;
        u8 is_data = 0;
        u8 reserved = 0, offset_to_data = 0;
@@ -180,7 +179,7 @@ static int wl1271_rx_handle_data(struct wl1271 *wl, u8 *data, u32 length,
         * packets copy the packets in offset of 2 bytes guarantee IP header
         * payload aligned to 4 bytes.
         */
-       buf = skb_put_data(skb, data + sizeof(*desc), pkt_data_len);
+       skb_put_data(skb, data + sizeof(*desc), pkt_data_len);
        if (rx_align == WLCORE_RX_BUF_PADDED)
                skb_pull(skb, RX_BUF_ALIGN);
 
index c8a8f5badb5bfc67a3fdf842754c8431282790de..c05cb637ba9234371b24d4f7ede54ff7b4a6f0c3 100644 (file)
@@ -1006,7 +1006,7 @@ static int pn533_start_poll_complete(struct pn533 *dev, struct sk_buff *resp)
 static struct sk_buff *pn533_alloc_poll_tg_frame(struct pn533 *dev)
 {
        struct sk_buff *skb;
-       u8 *felica, *nfcid3, *gb;
+       u8 *felica, *nfcid3;
 
        u8 *gbytes = dev->gb;
        size_t gbytes_len = dev->gb_len;
@@ -1048,7 +1048,7 @@ static struct sk_buff *pn533_alloc_poll_tg_frame(struct pn533 *dev)
        /* General bytes */
        skb_put_u8(skb, gbytes_len);
 
-       gb = skb_put_data(skb, gbytes, gbytes_len);
+       skb_put_data(skb, gbytes, gbytes_len);
 
        /* Len Tk */
        skb_put_u8(skb, 0);
index 14173cf6e1e76ff173f068dd5ba024b96aeae0e8..afb9dadc1cfe9fdeda30e2dbdb466a60343a0468 100644 (file)
@@ -1510,7 +1510,6 @@ static int amsdu_to_msdu(struct adapter *padapter, struct recv_frame *prframe)
        u8      nr_subframes, i;
        unsigned char *pdata;
        struct rx_pkt_attrib *pattrib;
-       unsigned char *data_ptr;
        struct sk_buff *sub_skb, *subframes[MAX_SUBFRAME_COUNT];
        struct recv_priv *precvpriv = &padapter->recvpriv;
        struct __queue *pfree_recv_queue = &(precvpriv->free_recv_queue);
@@ -1544,8 +1543,7 @@ static int amsdu_to_msdu(struct adapter *padapter, struct recv_frame *prframe)
                sub_skb = dev_alloc_skb(nSubframe_Length + 12);
                if (sub_skb) {
                        skb_reserve(sub_skb, 12);
-                       data_ptr = skb_put_data(sub_skb, pdata,
-                                               nSubframe_Length);
+                       skb_put_data(sub_skb, pdata, nSubframe_Length);
                } else {
                        sub_skb = skb_clone(prframe->pkt, GFP_ATOMIC);
                        if (sub_skb) {
index bae98ca0a9b6df292279b9e9e9df93b95cb1266d..03a81ba136b2261a0337aabd93218f3e3219e275 100644 (file)
@@ -782,7 +782,6 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb,
        u8              nPadding_Length = 0;
        u16             SeqNum = 0;
        struct sk_buff *sub_skb;
-       u8           *data_ptr;
        /* just for debug purpose */
        SeqNum = WLAN_GET_SEQ_SEQ(le16_to_cpu(hdr->seq_ctl));
        if ((RTLLIB_QOS_HAS_SEQ(fc)) &&
@@ -817,7 +816,7 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb,
                if (!sub_skb)
                        return 0;
                skb_reserve(sub_skb, 12);
-               data_ptr = skb_put_data(sub_skb, skb->data, skb->len);
+               skb_put_data(sub_skb, skb->data, skb->len);
                sub_skb->dev = ieee->dev;
 
                rxb->subframes[0] = sub_skb;
@@ -869,7 +868,7 @@ static u8 parse_subframe(struct rtllib_device *ieee, struct sk_buff *skb,
                if (!sub_skb)
                        return 0;
                skb_reserve(sub_skb, 12);
-               data_ptr = skb_put_data(sub_skb, skb->data, nSubframe_Length);
+               skb_put_data(sub_skb, skb->data, nSubframe_Length);
 
                sub_skb->dev = ieee->dev;
                rxb->subframes[rxb->nr_subframes++] = sub_skb;
index 5f2751d4d4641e37affdff910dd8f5afbdff8676..09d2c8649171c62d2e483840e2a121b241d9e37b 100644 (file)
@@ -1264,7 +1264,7 @@ rtllib_association_req(struct rtllib_network *beacon,
        hdr->info_element[0].id = MFIE_TYPE_SSID;
 
        hdr->info_element[0].len = beacon->ssid_len;
-       tag = skb_put_data(skb, beacon->ssid, beacon->ssid_len);
+       skb_put_data(skb, beacon->ssid, beacon->ssid_len);
 
        tag = skb_put(skb, rate_len);
 
@@ -1340,7 +1340,7 @@ rtllib_association_req(struct rtllib_network *beacon,
        }
 
        if (wpa_ie_len) {
-               tag = skb_put_data(skb, ieee->wpa_ie, ieee->wpa_ie_len);
+               skb_put_data(skb, ieee->wpa_ie, ieee->wpa_ie_len);
 
                if (PMKCacheIdx >= 0) {
                        tag = skb_put(skb, 18);
@@ -1356,12 +1356,13 @@ rtllib_association_req(struct rtllib_network *beacon,
        }
 
        if (wps_ie_len && ieee->wps_ie) {
-               tag = skb_put_data(skb, ieee->wps_ie, wps_ie_len);
+               skb_put_data(skb, ieee->wps_ie, wps_ie_len);
        }
 
-       tag = skb_put(skb, turbo_info_len);
-       if (turbo_info_len)
+       if (turbo_info_len) {
+               tag = skb_put(skb, turbo_info_len);
                rtllib_TURBO_Info(ieee, &tag);
+       }
 
        if (ieee->pHTInfo->bCurrentHTSupport && ieee->pHTInfo->bEnableHT) {
                if (ieee->pHTInfo->ePeerHTSpecVer == HT_SPEC_VER_EWC) {
index c0e2f711cb4ef2464cc450f1f88711586ef62e18..a4aedb489e927c7ca4551d2bba898ee516c37403 100644 (file)
@@ -780,7 +780,6 @@ static u8 parse_subframe(struct sk_buff *skb,
        u16             SeqNum=0;
 
        struct sk_buff *sub_skb;
-       u8             *data_ptr;
        /* just for debug purpose */
        SeqNum = WLAN_GET_SEQ_SEQ(le16_to_cpu(hdr->seq_ctl));
 
@@ -848,8 +847,7 @@ static u8 parse_subframe(struct sk_buff *skb,
                        if (!sub_skb)
                                return 0;
                        skb_reserve(sub_skb, 12);
-                       data_ptr = skb_put_data(sub_skb, skb->data,
-                                               nSubframe_Length);
+                       skb_put_data(sub_skb, skb->data, nSubframe_Length);
 #endif
                        rxb->subframes[rxb->nr_subframes++] = sub_skb;
                        if (rxb->nr_subframes >= MAX_SUBFRAME_COUNT) {
index 107069180ed249fe52579f009423c6604de617e6..fe6f38b7ec35864b001f593a7ff252868c3ebf4b 100644 (file)
@@ -1112,7 +1112,7 @@ ieee80211_association_req(struct ieee80211_network *beacon,
        hdr->info_element[0].id = MFIE_TYPE_SSID;
 
        hdr->info_element[0].len = beacon->ssid_len;
-       tag = skb_put_data(skb, beacon->ssid, beacon->ssid_len);
+       skb_put_data(skb, beacon->ssid, beacon->ssid_len);
 
        tag = skb_put(skb, rate_len);
 
@@ -1184,18 +1184,17 @@ ieee80211_association_req(struct ieee80211_network *beacon,
 
 
        //choose what wpa_supplicant gives to associate.
-       tag = skb_put(skb, wpa_ie_len);
        if (wpa_ie_len) {
-               memcpy(tag, ieee->wpa_ie, ieee->wpa_ie_len);
+               skb_put_data(skb, ieee->wpa_ie, wpa_ie_len);
        }
 
-       tag = skb_put(skb, wmm_info_len);
        if (wmm_info_len) {
-         ieee80211_WMM_Info(ieee, &tag);
+               tag = skb_put(skb, wmm_info_len);
+               ieee80211_WMM_Info(ieee, &tag);
        }
 #ifdef THOMAS_TURBO
-       tag = skb_put(skb, turbo_info_len);
        if (turbo_info_len) {
+               tag = skb_put(skb, turbo_info_len);
                ieee80211_TURBO_Info(ieee, &tag);
        }
 #endif
index c3cf01c842a3fa130bc4b9fc7c1f0970cb550386..87ab3ba760fcdd54c31798c4eca950ad349146d9 100644 (file)
@@ -31,7 +31,6 @@ rt_status SendTxCommandPacket(struct net_device *dev, void *pData, u32 DataLen)
        struct r8192_priv   *priv = ieee80211_priv(dev);
        struct sk_buff      *skb;
        struct cb_desc      *tcb_desc;
-       unsigned char       *ptr_buf;
 
        /* Get TCB and local buffer from common pool.
         * (It is shared by CmdQ, MgntQ, and USB coalesce DataQ)
@@ -45,7 +44,7 @@ rt_status SendTxCommandPacket(struct net_device *dev, void *pData, u32 DataLen)
        tcb_desc->bCmdOrInit = DESC_PACKET_TYPE_NORMAL;
        tcb_desc->bLastIniPkt = 0;
        skb_reserve(skb, USB_HWDESC_HEADER_LEN);
-       ptr_buf = skb_put_data(skb, pData, DataLen);
+       skb_put_data(skb, pData, DataLen);
        tcb_desc->txbuf_size = (u16)DataLen;
 
        if (!priv->ieee80211->check_nic_enough_desc(dev, tcb_desc->queue_index) ||
index f96c558b3c6ad5e04cd83ce1ec5a89d33f7b5d52..ea3eb94b28b314d23c944b5347fc8eee8e216405 100644 (file)
@@ -340,7 +340,7 @@ static int amsdu_to_msdu(struct _adapter *padapter, union recv_frame *prframe)
        int     a_len, padding_len;
        u16     eth_type, nSubframe_Length;
        u8      nr_subframes, i;
-       unsigned char *data_ptr, *pdata;
+       unsigned char *pdata;
        struct rx_pkt_attrib *pattrib;
        _pkt *sub_skb, *subframes[MAX_SUBFRAME_COUNT];
        struct recv_priv *precvpriv = &padapter->recvpriv;
@@ -372,7 +372,7 @@ static int amsdu_to_msdu(struct _adapter *padapter, union recv_frame *prframe)
                if (!sub_skb)
                        break;
                skb_reserve(sub_skb, 12);
-               data_ptr = skb_put_data(sub_skb, pdata, nSubframe_Length);
+               skb_put_data(sub_skb, pdata, nSubframe_Length);
                subframes[nr_subframes++] = sub_skb;
                if (nr_subframes >= MAX_SUBFRAME_COUNT) {
                        netdev_warn(padapter->pnetdev, "r8712u: ParseSubframe(): Too many Subframes! Packets dropped!\n");
index 1a6443dc3ff015db19aaa0049e23d1f430bbfb95..f42e00081e0e9724937f6db9eae947479b9d7d25 100644 (file)
@@ -72,7 +72,6 @@ int rtw_os_recvbuf_resource_free(struct adapter *padapter, struct recv_buf *prec
 _pkt *rtw_os_alloc_msdu_pkt(union recv_frame *prframe, u16 nSubframe_Length, u8 *pdata)
 {
        u16 eth_type;
-       u8 *data_ptr;
        _pkt *sub_skb;
        struct rx_pkt_attrib *pattrib;
 
@@ -82,8 +81,7 @@ _pkt *rtw_os_alloc_msdu_pkt(union recv_frame *prframe, u16 nSubframe_Length, u8
        if (sub_skb)
        {
                skb_reserve(sub_skb, 12);
-               data_ptr = skb_put_data(sub_skb, (pdata + ETH_HLEN),
-                                       nSubframe_Length);
+               skb_put_data(sub_skb, (pdata + ETH_HLEN), nSubframe_Length);
        }
        else
        {
index 1de67f209f2ca3b359c0e8678a2d0f6b950413a1..83ea8ab4f2f4c0dce2f46a2aba17209d71aadfa7 100644 (file)
@@ -3530,7 +3530,7 @@ static void hfa384x_int_rxmonitor(struct wlandevice *wlandev,
        /* Copy the 802.11 header to the skb
         * (ctl frames may be less than a full header)
         */
-       datap = skb_put_data(skb, &rxdesc->frame_control, hdrlen);
+       skb_put_data(skb, &rxdesc->frame_control, hdrlen);
 
        /* If any, copy the data from the card to the skb */
        if (datalen > 0) {
index a9c28c72c1c789a352c899b87f0c23519bc9cd10..24e34cfcb4bd9104b444c47274bf63ca7c868359 100644 (file)
@@ -1004,16 +1004,15 @@ static struct sk_buff *package_for_tx(struct f_ncm *ncm)
        }
 
        /* Insert NDP alignment. */
-       ntb_iter = skb_put_zero(skb2, ndp_pad);
+       skb_put_zero(skb2, ndp_pad);
 
        /* Copy NTB across. */
-       ntb_iter = skb_put_data(skb2, ncm->skb_tx_ndp->data,
-                               ncm->skb_tx_ndp->len);
+       skb_put_data(skb2, ncm->skb_tx_ndp->data, ncm->skb_tx_ndp->len);
        dev_consume_skb_any(ncm->skb_tx_ndp);
        ncm->skb_tx_ndp = NULL;
 
        /* Insert zero'd datagram. */
-       ntb_iter = skb_put_zero(skb2, dgram_idx_len);
+       skb_put_zero(skb2, dgram_idx_len);
 
        return skb2;
 }
@@ -1127,8 +1126,8 @@ static struct sk_buff *ncm_wrap_ntb(struct gether *port,
                ncm->ndp_dgram_count++;
 
                /* Add the new data to the skb */
-               ntb_data = skb_put_zero(ncm->skb_tx_data, dgram_pad);
-               ntb_data = skb_put_data(ncm->skb_tx_data, skb->data, skb->len);
+               skb_put_zero(ncm->skb_tx_data, dgram_pad);
+               skb_put_data(ncm->skb_tx_data, skb->data, skb->len);
                dev_consume_skb_any(skb);
                skb = NULL;
 
index a0c738aa6a79b94b73bcc25f82fad72a98ac0dee..fb52435be42dda3a5e7041b3554712bc80b4caad 100644 (file)
@@ -398,11 +398,7 @@ EXPORT_SYMBOL(__nla_reserve_64bit);
  */
 void *__nla_reserve_nohdr(struct sk_buff *skb, int attrlen)
 {
-       void *start;
-
-       start = skb_put_zero(skb, NLA_ALIGN(attrlen));
-
-       return start;
+       return skb_put_zero(skb, NLA_ALIGN(attrlen));
 }
 EXPORT_SYMBOL(__nla_reserve_nohdr);
 
index a05775afa44bbcf914d9410231493c345ffd0a26..eaf05de37f75ca204ded7efdba35c5257e8717d1 100644 (file)
@@ -107,7 +107,6 @@ static void nft_reject_br_send_v4_unreach(struct net *net,
        struct iphdr *niph;
        struct icmphdr *icmph;
        unsigned int len;
-       void *payload;
        __wsum csum;
        u8 proto;
 
@@ -151,7 +150,7 @@ static void nft_reject_br_send_v4_unreach(struct net *net,
        icmph->type     = ICMP_DEST_UNREACH;
        icmph->code     = code;
 
-       payload = skb_put_data(nskb, skb_network_header(oldskb), len);
+       skb_put_data(nskb, skb_network_header(oldskb), len);
 
        csum = csum_partial((void *)icmph, len + sizeof(struct icmphdr), 0);
        icmph->checksum = csum_fold(csum);
@@ -247,7 +246,6 @@ static void nft_reject_br_send_v6_unreach(struct net *net,
        struct ipv6hdr *nip6h;
        struct icmp6hdr *icmp6h;
        unsigned int len;
-       void *payload;
 
        if (!nft_bridge_ip6hdr_validate(oldskb))
                return;
@@ -277,7 +275,7 @@ static void nft_reject_br_send_v6_unreach(struct net *net,
        icmp6h->icmp6_type = ICMPV6_DEST_UNREACH;
        icmp6h->icmp6_code = code;
 
-       payload = skb_put_data(nskb, skb_network_header(oldskb), len);
+       skb_put_data(nskb, skb_network_header(oldskb), len);
        nip6h->payload_len = htons(nskb->len - sizeof(struct ipv6hdr));
 
        icmp6h->icmp6_cksum =
index 172f1316789622487214678cde9b628a9c85f550..b09e56214005c7cf9e257eb777882b234a2d6f18 100644 (file)
@@ -43,8 +43,7 @@ static struct sk_buff *trailer_xmit(struct sk_buff *skb, struct net_device *dev)
        kfree_skb(skb);
 
        if (padlen) {
-               u8 *pad = skb_put(nskb, padlen);
-               memset(pad, 0, padlen);
+               skb_put_zero(nskb, padlen);
        }
 
        trailer = skb_put(nskb, 4);
index 0a0a392dc2bd64b8c4202cc1361d828f9f984dd1..4e7bdb213cd076e44b5a36b2168413682bb23f68 100644 (file)
@@ -284,12 +284,12 @@ static void send_hsr_supervision_frame(struct hsr_port *master,
        skb_reset_mac_header(skb);
 
        if (hsrVer > 0) {
-               hsr_tag = (typeof(hsr_tag)) skb_put(skb, sizeof(struct hsr_tag));
+               hsr_tag = skb_put(skb, sizeof(struct hsr_tag));
                hsr_tag->encap_proto = htons(ETH_P_PRP);
                set_hsr_tag_LSDU_size(hsr_tag, HSR_V1_SUP_LSDUSIZE);
        }
 
-       hsr_stag = (typeof(hsr_stag)) skb_put(skb, sizeof(struct hsr_sup_tag));
+       hsr_stag = skb_put(skb, sizeof(struct hsr_sup_tag));
        set_hsr_stag_path(hsr_stag, (hsrVer ? 0x0 : 0xf));
        set_hsr_stag_HSR_Ver(hsr_stag, hsrVer);
 
@@ -311,7 +311,7 @@ static void send_hsr_supervision_frame(struct hsr_port *master,
        hsr_stag->HSR_TLV_Length = hsrVer ? sizeof(struct hsr_sup_payload) : 12;
 
        /* Payload: MacAddressA */
-       hsr_sp = (typeof(hsr_sp)) skb_put(skb, sizeof(struct hsr_sup_payload));
+       hsr_sp = skb_put(skb, sizeof(struct hsr_sup_payload));
        ether_addr_copy(hsr_sp->MacAddressA, master->dev->dev_addr);
 
        skb_put_padto(skb, ETH_ZLEN + HSR_HLEN);
index 82e71e5622c28b448bfb5a340b917c832a9770d1..debda3de4726a8e4a6dc571143b5cdc24306a1a0 100644 (file)
@@ -392,7 +392,7 @@ void irlap_send_discovery_xid_frame(struct irlap_cb *self, int S, __u8 s,
                info[0] = discovery->data.charset;
 
                len = IRDA_MIN(discovery->name_len, skb_tailroom(tx_skb));
-               info = skb_put_data(tx_skb, discovery->data.info, len);
+               skb_put_data(tx_skb, discovery->data.info, len);
        }
        irlap_queue_xmit(self, tx_skb);
 }
@@ -1194,7 +1194,6 @@ void irlap_send_test_frame(struct irlap_cb *self, __u8 caddr, __u32 daddr,
 {
        struct sk_buff *tx_skb;
        struct test_frame *frame;
-       __u8 *info;
 
        tx_skb = alloc_skb(cmd->len + sizeof(struct test_frame), GFP_ATOMIC);
        if (!tx_skb)
@@ -1214,7 +1213,7 @@ void irlap_send_test_frame(struct irlap_cb *self, __u8 caddr, __u32 daddr,
        frame->control = TEST_RSP | PF_BIT;
 
        /* Copy info */
-       info = skb_put_data(tx_skb, cmd->data, cmd->len);
+       skb_put_data(tx_skb, cmd->data, cmd->len);
 
        /* Return to sender */
        irlap_wait_min_turn_around(self, &self->qos_tx);
index 7be7917e154192cac641988a2eba25074d94560e..b588e593b0ec175a72251345174928b724e83838 100644 (file)
@@ -796,8 +796,7 @@ static void ieee80211_send_assoc(struct ieee80211_sub_if_data *sdata)
                                                 after_ric,
                                                 ARRAY_SIZE(after_ric),
                                                 offset);
-               pos = skb_put_data(skb, assoc_data->ie + offset,
-                                  noffset - offset);
+               skb_put_data(skb, assoc_data->ie + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -834,8 +833,7 @@ static void ieee80211_send_assoc(struct ieee80211_sub_if_data *sdata)
                noffset = ieee80211_ie_split(assoc_data->ie, assoc_data->ie_len,
                                             before_vht, ARRAY_SIZE(before_vht),
                                             offset);
-               pos = skb_put_data(skb, assoc_data->ie + offset,
-                                  noffset - offset);
+               skb_put_data(skb, assoc_data->ie + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -848,8 +846,7 @@ static void ieee80211_send_assoc(struct ieee80211_sub_if_data *sdata)
                noffset = ieee80211_ie_split_vendor(assoc_data->ie,
                                                    assoc_data->ie_len,
                                                    offset);
-               pos = skb_put_data(skb, assoc_data->ie + offset,
-                                  noffset - offset);
+               skb_put_data(skb, assoc_data->ie + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -868,8 +865,7 @@ static void ieee80211_send_assoc(struct ieee80211_sub_if_data *sdata)
        /* add any remaining custom (i.e. vendor specific here) IEs */
        if (assoc_data->ie_len) {
                noffset = assoc_data->ie_len;
-               pos = skb_put_data(skb, assoc_data->ie + offset,
-                                  noffset - offset);
+               skb_put_data(skb, assoc_data->ie + offset, noffset - offset);
        }
 
        if (assoc_data->fils_kek_len &&
index 709ef02fe67e5490c6b04dcfb233067994295d06..91093d4a2f841bb3f1987cde6d9237dd406d1f02 100644 (file)
@@ -388,7 +388,7 @@ ieee80211_tdls_add_setup_start_ies(struct ieee80211_sub_if_data *sdata,
                                             before_ext_cap,
                                             ARRAY_SIZE(before_ext_cap),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -417,7 +417,7 @@ ieee80211_tdls_add_setup_start_ies(struct ieee80211_sub_if_data *sdata,
                                             before_ht_cap,
                                             ARRAY_SIZE(before_ht_cap),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -488,7 +488,7 @@ ieee80211_tdls_add_setup_start_ies(struct ieee80211_sub_if_data *sdata,
                                             before_vht_cap,
                                             ARRAY_SIZE(before_vht_cap),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -529,7 +529,7 @@ ieee80211_tdls_add_setup_start_ies(struct ieee80211_sub_if_data *sdata,
        /* add any remaining IEs */
        if (extra_ies_len) {
                noffset = extra_ies_len;
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
        }
 
 }
@@ -571,7 +571,7 @@ ieee80211_tdls_add_setup_cfm_ies(struct ieee80211_sub_if_data *sdata,
                                             before_qos,
                                             ARRAY_SIZE(before_qos),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -591,7 +591,7 @@ ieee80211_tdls_add_setup_cfm_ies(struct ieee80211_sub_if_data *sdata,
                                             before_ht_op,
                                             ARRAY_SIZE(before_ht_op),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -632,7 +632,7 @@ ieee80211_tdls_add_setup_cfm_ies(struct ieee80211_sub_if_data *sdata,
        /* add any remaining IEs */
        if (extra_ies_len) {
                noffset = extra_ies_len;
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
        }
 }
 
@@ -645,7 +645,6 @@ ieee80211_tdls_add_chan_switch_req_ies(struct ieee80211_sub_if_data *sdata,
 {
        struct ieee80211_tdls_data *tf;
        size_t offset = 0, noffset;
-       u8 *pos;
 
        if (WARN_ON_ONCE(!chandef))
                return;
@@ -663,7 +662,7 @@ ieee80211_tdls_add_chan_switch_req_ies(struct ieee80211_sub_if_data *sdata,
                                             before_lnkie,
                                             ARRAY_SIZE(before_lnkie),
                                             offset);
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
                offset = noffset;
        }
 
@@ -672,7 +671,7 @@ ieee80211_tdls_add_chan_switch_req_ies(struct ieee80211_sub_if_data *sdata,
        /* add any remaining IEs */
        if (extra_ies_len) {
                noffset = extra_ies_len;
-               pos = skb_put_data(skb, extra_ies + offset, noffset - offset);
+               skb_put_data(skb, extra_ies + offset, noffset - offset);
        }
 }
 
index 2c196b3e9cd3d1b1a7648917446b927f5f35cbf0..4b1967997c1650a94d49bdca5c4eaa279e9220bf 100644 (file)
@@ -1474,11 +1474,10 @@ void sctp_chunk_put(struct sctp_chunk *ch)
 void *sctp_addto_chunk(struct sctp_chunk *chunk, int len, const void *data)
 {
        void *target;
-       void *padding;
        int chunklen = ntohs(chunk->chunk_hdr->length);
        int padlen = SCTP_PAD4(chunklen) - chunklen;
 
-       padding = skb_put_zero(chunk->skb, padlen);
+       skb_put_zero(chunk->skb, padlen);
        target = skb_put_data(chunk->skb, data, len);
 
        /* Adjust the chunk length field.  */
index 7d6ee03f276223c8f2674641cb617a10bbd0958f..edba7ab975639fc08c4257d0393391477979b1aa 100644 (file)
@@ -90,7 +90,6 @@ out_pkt:
 static struct sk_buff *virtio_transport_build_skb(void *opaque)
 {
        struct virtio_vsock_pkt *pkt = opaque;
-       unsigned char *t_hdr, *payload;
        struct af_vsockmon_hdr *hdr;
        struct sk_buff *skb;
 
@@ -132,10 +131,10 @@ static struct sk_buff *virtio_transport_build_skb(void *opaque)
                break;
        }
 
-       t_hdr = skb_put_data(skb, &pkt->hdr, sizeof(pkt->hdr));
+       skb_put_data(skb, &pkt->hdr, sizeof(pkt->hdr));
 
        if (pkt->len) {
-               payload = skb_put_data(skb, pkt->buf, pkt->len);
+               skb_put_data(skb, pkt->buf, pkt->len);
        }
 
        return skb;
index eb466ece1730998550ba65d3cf65fc8fdf363d59..db0b1315d577aa72d2bf53bac0c78cc87e7a67ec 100644 (file)
@@ -188,14 +188,14 @@ void x25_write_internal(struct sock *sk, int frametype)
                        *dptr++ = X25_CALL_REQUEST;
                        len     = x25_addr_aton(addresses, &x25->dest_addr,
                                                &x25->source_addr);
-                       dptr = skb_put_data(skb, addresses, len);
+                       skb_put_data(skb, addresses, len);
                        len     = x25_create_facilities(facilities,
                                        &x25->facilities,
                                        &x25->dte_facilities,
                                        x25->neighbour->global_facil_mask);
-                       dptr = skb_put_data(skb, facilities, len);
-                       dptr = skb_put_data(skb, x25->calluserdata.cuddata,
-                                           x25->calluserdata.cudlength);
+                       skb_put_data(skb, facilities, len);
+                       skb_put_data(skb, x25->calluserdata.cuddata,
+                                    x25->calluserdata.cudlength);
                        x25->calluserdata.cudlength = 0;
                        break;
 
@@ -207,15 +207,15 @@ void x25_write_internal(struct sock *sk, int frametype)
                                                        &x25->facilities,
                                                        &x25->dte_facilities,
                                                        x25->vc_facil_mask);
-                       dptr = skb_put_data(skb, facilities, len);
+                       skb_put_data(skb, facilities, len);
 
                        /* fast select with no restriction on response
                                allows call user data. Userland must
                                ensure it is ours and not theirs */
                        if(x25->facilities.reverse & 0x80) {
-                               dptr = skb_put_data(skb,
-                                                   x25->calluserdata.cuddata,
-                                                   x25->calluserdata.cudlength);
+                               skb_put_data(skb,
+                                            x25->calluserdata.cuddata,
+                                            x25->calluserdata.cudlength);
                        }
                        x25->calluserdata.cudlength = 0;
                        break;