Don't omit m_dup() for non-writeable mbufs that need checksum calculation

[zbb]

Summary

If the driver is not active or link is down the packet could remain
non-writeable. This commit makes all mbufs enqueued to the driver's
ring buffer to have correct attributes.

Pointed out by: wma
Reviewed by:
Obtained from: Semihalf
Sponsored by: Cavium
Differential Revision:

head/sys/dev/vnic/nicvf_main.c

TAILQ_FOREACH(ifp, &V_ifnet, if_link) {
                 IF_ADDR_RLOCK(ifp);
                 TAILQ_FOREACH(ifa, &ifp->if_addrhead, ifa_link) {
                         struct sockaddr *sa = ifa->ifa_addr;
 
                         if (prison_if(curthread->td_ucred, sa) != 0)
                                 continue;
                         addrs++;
                         if (sa->sa_len <= sizeof(*sa)) {
-                                ifr.ifr_addr = *sa;
+                                if (sa->sa_len < sizeof(*sa)) {
+                                        memset(&ifr.ifr_ifru.ifru_addr, 0,
+                                            sizeof(ifr.ifr_ifru.ifru_addr));
+                                        memcpy(&ifr.ifr_ifru.ifru_addr, sa,
+                                            sa->sa_len);
+                                } else
+                                        ifr.ifr_ifru.ifru_addr = *sa;
                                 sbuf_bcat(sb, &ifr, sizeof(ifr));
                                 max_len += sizeof(ifr);
                         } else {
                                 sbuf_bcat(sb, &ifr,
                                     offsetof(struct ifreq, ifr_addr));
                                 max_len += offsetof(struct ifreq, ifr_addr);
                                 sbuf_bcat(sb, sa, sa->sa_len);
                                 max_len += sa->sa_len;