Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net

Pull more networking fixes from David Miller:
 "Some more bug fixes have trickled in, we have:

  1) Local MAC entries properly in mscc driver, from Allan W. Nielsen.

  2) Eric Dumazet found some more of the typical "pskb_may_pull() -->
     oops forgot to reload the header pointer" bugs in ipv6 tunnel
     handling.

  3) Bad SKB socket pointer in ipv6 fragmentation handling, from Herbert
     Xu.

  4) Overflow fix in sk_msg_clone(), from Vakul Garg.

  5) Validate address lengths in AF_PACKET, from Willem de Bruijn"

* git://git.kernel.org/pub/scm/linux/kernel/git/davem/net:
  qmi_wwan: Fix qmap header retrieval in qmimux_rx_fixup
  qmi_wwan: Add support for Fibocom NL678 series
  tls: Do not call sk_memcopy_from_iter with zero length
  ipv6: tunnels: fix two use-after-free
  Prevent overflow of sk_msg in sk_msg_clone()
  packet: validate address length
  net: netxen: fix a missing check and an uninitialized use
  tcp: fix a race in inet_diag_dump_icsk()
  MAINTAINERS: update cxgb4 and cxgb3 maintainer
  ipv6: frags: Fix bogus skb->sk in reassembled packets
  mscc: Configured MAC entries should be locked.

11 files changed, 33 insertions, 16 deletions

diff --git a/MAINTAINERS b/MAINTAINERS
index 842b697a1511..f3a5c97e3419 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4057,7 +4057,7 @@ S:	Maintained
 F:      drivers/media/dvb-frontends/cxd2820r*
 
 CXGB3 ETHERNET DRIVER (CXGB3)
-M:      Santosh Raspatur <santosh@chelsio.com>
+M:      Arjun Vynipadath <arjun@chelsio.com>
 L:      netdev@vger.kernel.org
 W:      http://www.chelsio.com
 S:      Supported
@@ -4086,7 +4086,7 @@ S:	Supported
 F:      drivers/crypto/chelsio
 
 CXGB4 ETHERNET DRIVER (CXGB4)
-M:      Ganesh Goudar <ganeshgr@chelsio.com>
+M:      Arjun Vynipadath <arjun@chelsio.com>
 L:      netdev@vger.kernel.org
 W:      http://www.chelsio.com
 S:      Supported
diff --git a/drivers/net/ethernet/mscc/ocelot.c b/drivers/net/ethernet/mscc/ocelot.c
index 3238b9ee42f3..c84074fa4c95 100644
--- a/drivers/net/ethernet/mscc/ocelot.c
+++ b/drivers/net/ethernet/mscc/ocelot.c
@@ -747,7 +747,7 @@ static int ocelot_fdb_add(struct ndmsg *ndm, struct nlattr *tb[],
         }
 
         return ocelot_mact_learn(ocelot, port->chip_port, addr, vid,
-                                 ENTRYTYPE_NORMAL);
+                                 ENTRYTYPE_LOCKED);
 }
 
 static int ocelot_fdb_del(struct ndmsg *ndm, struct nlattr *tb[],
diff --git a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
index 0ea141ece19e..6547a9dd5935 100644
--- a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
+++ b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c
@@ -1125,7 +1125,8 @@ netxen_validate_firmware(struct netxen_adapter *adapter)
                 return -EINVAL;
         }
         val = nx_get_bios_version(adapter);
-        netxen_rom_fast_read(adapter, NX_BIOS_VERSION_OFFSET, (int *)&bios);
+        if (netxen_rom_fast_read(adapter, NX_BIOS_VERSION_OFFSET, (int *)&bios))
+                return -EIO;
         if ((__force u32)val != bios) {
                 dev_err(&pdev->dev, "%s: firmware bios is incompatible\n",
                                 fw_name[fw_type]);
diff --git a/drivers/net/usb/qmi_wwan.c b/drivers/net/usb/qmi_wwan.c
index c8872dd5ff5e..774e1ff01c9a 100644
--- a/drivers/net/usb/qmi_wwan.c
+++ b/drivers/net/usb/qmi_wwan.c
@@ -151,17 +151,18 @@ static bool qmimux_has_slaves(struct usbnet *dev)
 
 static int qmimux_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
 {
-        unsigned int len, offset = sizeof(struct qmimux_hdr);
+        unsigned int len, offset = 0;
         struct qmimux_hdr *hdr;
         struct net_device *net;
         struct sk_buff *skbn;
+        u8 qmimux_hdr_sz = sizeof(*hdr);
 
-        while (offset < skb->len) {
-                hdr = (struct qmimux_hdr *)skb->data;
+        while (offset + qmimux_hdr_sz < skb->len) {
+                hdr = (struct qmimux_hdr *)(skb->data + offset);
                 len = be16_to_cpu(hdr->pkt_len);
 
                 /* drop the packet, bogus length */
-                if (offset + len > skb->len)
+                if (offset + len + qmimux_hdr_sz > skb->len)
                         return 0;
 
                 /* control packet, we do not know what to do */
@@ -176,7 +177,7 @@ static int qmimux_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
                         return 0;
                 skbn->dev = net;
 
-                switch (skb->data[offset] & 0xf0) {
+                switch (skb->data[offset + qmimux_hdr_sz] & 0xf0) {
                 case 0x40:
                         skbn->protocol = htons(ETH_P_IP);
                         break;
@@ -188,12 +189,12 @@ static int qmimux_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
                         goto skip;
                 }
 
-                skb_put_data(skbn, skb->data + offset, len);
+                skb_put_data(skbn, skb->data + offset + qmimux_hdr_sz, len);
                 if (netif_rx(skbn) != NET_RX_SUCCESS)
                         return 0;
 
 skip:
-                offset += len + sizeof(struct qmimux_hdr);
+                offset += len + qmimux_hdr_sz;
         }
         return 1;
 }
@@ -1265,6 +1266,7 @@ static const struct usb_device_id products[] = {
         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0121, 4)}, /* Quectel EC21 Mini PCIe */
         {QMI_QUIRK_SET_DTR(0x2c7c, 0x0191, 4)}, /* Quectel EG91 */
         {QMI_FIXED_INTF(0x2c7c, 0x0296, 4)},    /* Quectel BG96 */
+        {QMI_QUIRK_SET_DTR(0x2cb7, 0x0104, 4)}, /* Fibocom NL678 series */
 
         /* 4. Gobi 1000 devices */
         {QMI_GOBI1K_DEVICE(0x05c6, 0x9212)},    /* Acer Gobi Modem Device */
diff --git a/net/core/skmsg.c b/net/core/skmsg.c
index 56a99d0c9aa0..b7dbb3c976cd 100644
--- a/net/core/skmsg.c
+++ b/net/core/skmsg.c
@@ -94,6 +94,9 @@ int sk_msg_clone(struct sock *sk, struct sk_msg *dst, struct sk_msg *src,
         }
 
         while (len) {
+                if (sk_msg_full(dst))
+                        return -ENOSPC;
+
                 sge_len = sge->length - off;
                 sge_off = sge->offset + off;
                 if (sge_len > len)
diff --git a/net/ipv4/inet_diag.c b/net/ipv4/inet_diag.c
index 4e5bc4b2f14e..1a4e9ff02762 100644
--- a/net/ipv4/inet_diag.c
+++ b/net/ipv4/inet_diag.c
@@ -998,7 +998,9 @@ next_chunk:
                         if (!inet_diag_bc_sk(bc, sk))
                                 goto next_normal;
 
-                        sock_hold(sk);
+                        if (!refcount_inc_not_zero(&sk->sk_refcnt))
+                                goto next_normal;
+
                         num_arr[accum] = num;
                         sk_arr[accum] = sk;
                         if (++accum == SKARR_SZ)
diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
index a9d06d4dd057..99179b9c8384 100644
--- a/net/ipv6/ip6_tunnel.c
+++ b/net/ipv6/ip6_tunnel.c
@@ -901,6 +901,7 @@ static int ipxip6_rcv(struct sk_buff *skb, u8 ipproto,
                         goto drop;
                 if (!xfrm6_policy_check(NULL, XFRM_POLICY_IN, skb))
                         goto drop;
+                ipv6h = ipv6_hdr(skb);
                 if (!ip6_tnl_rcv_ctl(t, &ipv6h->daddr, &ipv6h->saddr))
                         goto drop;
                 if (iptunnel_pull_header(skb, 0, tpi->proto, false))
diff --git a/net/ipv6/ip6_vti.c b/net/ipv6/ip6_vti.c
index eeaf7455d51e..706fe42e4928 100644
--- a/net/ipv6/ip6_vti.c
+++ b/net/ipv6/ip6_vti.c
@@ -318,6 +318,7 @@ static int vti6_rcv(struct sk_buff *skb)
                         return 0;
                 }
 
+                ipv6h = ipv6_hdr(skb);
                 if (!ip6_tnl_rcv_ctl(t, &ipv6h->daddr, &ipv6h->saddr)) {
                         t->dev->stats.rx_dropped++;
                         rcu_read_unlock();
diff --git a/net/ipv6/reassembly.c b/net/ipv6/reassembly.c
index aa26c45486d9..a5bb59ee50ac 100644
--- a/net/ipv6/reassembly.c
+++ b/net/ipv6/reassembly.c
@@ -384,6 +384,7 @@ static int ip6_frag_reasm(struct frag_queue *fq, struct sk_buff *prev,
                 if (skb_try_coalesce(head, fp, &headstolen, &delta)) {
                         kfree_skb_partial(fp, headstolen);
                 } else {
+                        fp->sk = NULL;
                         if (!skb_shinfo(head)->frag_list)
                                 skb_shinfo(head)->frag_list = fp;
                         head->data_len += fp->len;
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c
index 6655793765b2..5dda263b4a0a 100644
--- a/net/packet/af_packet.c
+++ b/net/packet/af_packet.c
@@ -2627,6 +2627,8 @@ static int tpacket_snd(struct packet_sock *po, struct msghdr *msg)
                 proto   = saddr->sll_protocol;
                 addr    = saddr->sll_addr;
                 dev = dev_get_by_index(sock_net(&po->sk), saddr->sll_ifindex);
+                if (addr && dev && saddr->sll_halen < dev->addr_len)
+                        goto out;
         }
 
         err = -ENXIO;
@@ -2825,6 +2827,8 @@ static int packet_snd(struct socket *sock, struct msghdr *msg, size_t len)
                 proto   = saddr->sll_protocol;
                 addr    = saddr->sll_addr;
                 dev = dev_get_by_index(sock_net(sk), saddr->sll_ifindex);
+                if (addr && dev && saddr->sll_halen < dev->addr_len)
+                        goto out;
         }
 
         err = -ENXIO;
diff --git a/net/tls/tls_sw.c b/net/tls/tls_sw.c
index 7b1af8b59cd2..29b27858fff1 100644
--- a/net/tls/tls_sw.c
+++ b/net/tls/tls_sw.c
@@ -935,10 +935,12 @@ fallback_to_reg_send:
                                     tls_ctx->tx.overhead_size);
                 }
 
-                ret = sk_msg_memcopy_from_iter(sk, &msg->msg_iter, msg_pl,
-                                               try_to_copy);
-                if (ret < 0)
-                        goto trim_sgl;
+                if (try_to_copy) {
+                        ret = sk_msg_memcopy_from_iter(sk, &msg->msg_iter,
+                                                       msg_pl, try_to_copy);
+                        if (ret < 0)
+                                goto trim_sgl;
+                }
 
                 /* Open records defined only if successfully copied, otherwise
                  * we would trim the sg but not reset the open record frags.