Prefer using simple structure assignment instead of memcpy. Using a structure assignment preserves type information and compiler checks types already.
Signed-off-by: Stephen Hemminger <step...@networkplumber.org> --- Reran cocci script against -rc1 and did minor replacement of "*&" drivers/common/sfc_efx/base/efx_mae.c | 2 +- drivers/net/zxdh/zxdh_ethdev_ops.c | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/common/sfc_efx/base/efx_mae.c b/drivers/common/sfc_efx/base/efx_mae.c index 1429e7dd0a..4533e5cae6 100644 --- a/drivers/common/sfc_efx/base/efx_mae.c +++ b/drivers/common/sfc_efx/base/efx_mae.c @@ -1353,7 +1353,7 @@ efx_mae_match_spec_clone( goto fail1; } - memcpy(clone, orig, sizeof (efx_mae_match_spec_t)); + *clone = *orig; *clonep = clone; diff --git a/drivers/net/zxdh/zxdh_ethdev_ops.c b/drivers/net/zxdh/zxdh_ethdev_ops.c index f8e8d26c50..268f78900c 100644 --- a/drivers/net/zxdh/zxdh_ethdev_ops.c +++ b/drivers/net/zxdh/zxdh_ethdev_ops.c @@ -444,7 +444,7 @@ zxdh_dev_mac_addr_set(struct rte_eth_dev *dev, struct rte_ether_addr *addr) } else { struct zxdh_mac_filter *mac_filter = &msg_info.data.mac_filter_msg; mac_filter->filter_flag = ZXDH_MAC_UNFILTER; - memcpy(&mac_filter->mac, addr, sizeof(struct rte_ether_addr)); + mac_filter->mac = *addr; zxdh_msg_head_build(hw, ZXDH_MAC_ADD, &msg_info); ret = zxdh_vf_send_msg_to_pf(dev, &msg_info, sizeof(msg_info), NULL, 0); if (ret) { @@ -460,7 +460,7 @@ zxdh_dev_mac_addr_set(struct rte_eth_dev *dev, struct rte_ether_addr *addr) mac_filter->filter_flag = ZXDH_MAC_UNFILTER; mac_filter->mac_flag = true; - memcpy(&mac_filter->mac, old_addr, sizeof(struct rte_ether_addr)); + mac_filter->mac = *old_addr; zxdh_msg_head_build(hw, ZXDH_MAC_DEL, &msg_info); ret = zxdh_vf_send_msg_to_pf(dev, &msg_info, sizeof(msg_info), NULL, 0); if (ret) { @@ -532,7 +532,7 @@ zxdh_dev_mac_addr_add(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr, struct zxdh_mac_filter *mac_filter = &msg_info.data.mac_filter_msg; mac_filter->filter_flag = ZXDH_MAC_FILTER; - memcpy(&mac_filter->mac, mac_addr, sizeof(struct rte_ether_addr)); + mac_filter->mac = *mac_addr; zxdh_msg_head_build(hw, ZXDH_MAC_ADD, &msg_info); if (rte_is_unicast_ether_addr(mac_addr)) { if (hw->uc_num < ZXDH_MAX_UC_MAC_ADDRS) { @@ -614,7 +614,7 @@ void zxdh_dev_mac_addr_remove(struct rte_eth_dev *dev, uint32_t index) struct zxdh_mac_filter *mac_filter = &msg_info.data.mac_filter_msg; mac_filter->filter_flag = ZXDH_MAC_FILTER; - memcpy(&mac_filter->mac, mac_addr, sizeof(struct rte_ether_addr)); + mac_filter->mac = *mac_addr; zxdh_msg_head_build(hw, ZXDH_MAC_DEL, &msg_info); if (rte_is_unicast_ether_addr(mac_addr)) { if (hw->uc_num <= ZXDH_MAX_UC_MAC_ADDRS) { -- 2.47.2