[PATCH v2 14/14] security: remove unnecessary dereference

Stephen Hemminger stephen at networkplumber.org
Thu Feb 20 03:32:18 CET 2025


The expressions *instance->ops->set_pkt_metadata and
instance->ops->set_pkt_metadata are equivalent in C.

Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
---
 lib/security/rte_security.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/security/rte_security.c b/lib/security/rte_security.c
index e5c862f5f5..1e980343fe 100644
--- a/lib/security/rte_security.c
+++ b/lib/security/rte_security.c
@@ -258,10 +258,9 @@ __rte_security_set_pkt_metadata(void *ctx, void *sess, struct rte_mbuf *m, void
 	RTE_PTR_OR_ERR_RET(instance, -EINVAL);
 	RTE_PTR_OR_ERR_RET(instance->ops, -EINVAL);
 #endif
-	if (*instance->ops->set_pkt_metadata == NULL)
+	if (instance->ops->set_pkt_metadata == NULL)
 		return -ENOTSUP;
-	return instance->ops->set_pkt_metadata(instance->device,
-					       sess, m, params);
+	return instance->ops->set_pkt_metadata(instance->device, sess, m, params);
 }
 
 const struct rte_security_capability *
-- 
2.47.2



More information about the dev mailing list