mirror of
https://github.com/apache/nuttx.git
synced 2026-05-20 20:44:39 +08:00
net:fix coverity warning
Signed-off-by: hejianliang3 <hejianliang3@xiaomi.com>
This commit is contained in:
committed by
Petro Karashchenko
parent
451c53daa4
commit
e9648d8a73
+1
-1
@@ -289,7 +289,7 @@ int arp_send(in_addr_t ipaddr)
|
||||
|
||||
/* Remember the routing device name */
|
||||
|
||||
strncpy((FAR char *)state.snd_ifname, (FAR const char *)dev->d_ifname,
|
||||
strlcpy((FAR char *)state.snd_ifname, (FAR const char *)dev->d_ifname,
|
||||
IFNAMSIZ);
|
||||
|
||||
/* Now loop, testing if the address mapping is in the ARP table and re-
|
||||
|
||||
@@ -122,7 +122,7 @@ static int ifconf_ipv4_callback(FAR struct net_driver_s *dev, FAR void *arg)
|
||||
* transferred is returned in ifc_len.
|
||||
*/
|
||||
|
||||
strncpy(req->ifr_name, dev->d_ifname, IFNAMSIZ);
|
||||
strlcpy(req->ifr_name, dev->d_ifname, IFNAMSIZ);
|
||||
|
||||
inaddr->sin_family = AF_INET;
|
||||
inaddr->sin_port = 0;
|
||||
|
||||
@@ -1086,7 +1086,7 @@ static int netdev_ifr_ioctl(FAR struct socket *psock, int cmd,
|
||||
dev = netdev_findbyindex(req->ifr_ifindex);
|
||||
if (dev != NULL)
|
||||
{
|
||||
strncpy(req->ifr_name, dev->d_ifname, IFNAMSIZ);
|
||||
strlcpy(req->ifr_name, dev->d_ifname, IFNAMSIZ);
|
||||
ret = OK;
|
||||
}
|
||||
else
|
||||
|
||||
@@ -399,7 +399,7 @@ int netdev_register(FAR struct net_driver_s *dev, enum net_lltype_e lltype)
|
||||
*/
|
||||
|
||||
dev->d_ifname[IFNAMSIZ - 1] = '\0';
|
||||
strncpy(devfmt_str, dev->d_ifname, IFNAMSIZ);
|
||||
strlcpy(devfmt_str, dev->d_ifname, IFNAMSIZ);
|
||||
|
||||
/* Then use the content of the temporary buffer as the format
|
||||
* string.
|
||||
|
||||
@@ -217,7 +217,7 @@ netlink_get_device(FAR struct net_driver_s *dev,
|
||||
resp->attr.rta_len = RTA_LENGTH(strnlen(dev->d_ifname, IFNAMSIZ));
|
||||
resp->attr.rta_type = IFLA_IFNAME;
|
||||
|
||||
strncpy((FAR char *)resp->data, dev->d_ifname, IFNAMSIZ);
|
||||
strlcpy((FAR char *)resp->data, dev->d_ifname, IFNAMSIZ);
|
||||
|
||||
/* Finally, return the response */
|
||||
|
||||
|
||||
@@ -517,8 +517,10 @@ static void rpmsg_socket_ns_bind(FAR struct rpmsg_device *rdev,
|
||||
return;
|
||||
}
|
||||
|
||||
strcpy(new->rpaddr.rp_cpu, rpmsg_get_cpuname(rdev));
|
||||
strcpy(new->rpaddr.rp_name, name + RPMSG_SOCKET_NAME_PREFIX_LEN);
|
||||
strlcpy(new->rpaddr.rp_cpu, rpmsg_get_cpuname(rdev),
|
||||
sizeof(new->rpaddr.rp_cpu));
|
||||
strlcpy(new->rpaddr.rp_name, name + RPMSG_SOCKET_NAME_PREFIX_LEN,
|
||||
sizeof(new->rpaddr.rp_name));
|
||||
|
||||
rpmsg_socket_lock(&server->recvlock);
|
||||
|
||||
|
||||
@@ -140,15 +140,7 @@ int psock_connect(FAR struct socket *psock, FAR const struct sockaddr *addr,
|
||||
if (ret >= 0)
|
||||
{
|
||||
FAR struct socket_conn_s *conn = psock->s_conn;
|
||||
|
||||
if (addr != NULL)
|
||||
{
|
||||
conn->s_flags |= _SF_CONNECTED;
|
||||
}
|
||||
else
|
||||
{
|
||||
conn->s_flags &= ~_SF_CONNECTED;
|
||||
}
|
||||
conn->s_flags |= _SF_CONNECTED;
|
||||
}
|
||||
|
||||
return ret;
|
||||
|
||||
@@ -96,7 +96,7 @@ uint16_t ipv4_upperlayer_chksum(FAR struct net_driver_s *dev, uint8_t proto)
|
||||
|
||||
/* Sum IP source and destination addresses. */
|
||||
|
||||
sum = chksum(sum, (FAR uint8_t *)&ipv4->srcipaddr, 2 * sizeof(in_addr_t));
|
||||
sum = chksum(sum, (FAR uint8_t *)&ipv4->srcipaddr, sizeof(in_addr_t));
|
||||
|
||||
/* Sum IP payload data. */
|
||||
|
||||
|
||||
Reference in New Issue
Block a user