From: Praveen Shetty Date: Wed, 6 May 2020 11:02:41 +0000 (+0100) Subject: examples/ipsec-secgw: fix ESP flow error log X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=5ec3eb3b839ded5f0f04754721475433bf5d69df;p=dpdk.git examples/ipsec-secgw: fix ESP flow error log Function create_ipsec_esp_flow returns a negative number in case of any failure and we are passing this to strerror to display the error message. But strerror()'s argument cannot be negative. In case of failure, displaying exact error message to console is handled in create_ipsec_esp_flow function. So it is not required to print the error message again using strerror. This patch will remove the unnecessary calling of strerror function to fix the negative argument passing to strerror issue. Coverity issue: 357691 Fixes: 6738c0a95695 ("examples/ipsec-secgw: support flow director") Signed-off-by: Praveen Shetty Acked-by: Lukasz Wojciechowski Acked-by: Anoob Joseph --- diff --git a/examples/ipsec-secgw/sa.c b/examples/ipsec-secgw/sa.c index e3a1a5affe..632482176c 100644 --- a/examples/ipsec-secgw/sa.c +++ b/examples/ipsec-secgw/sa.c @@ -1223,8 +1223,7 @@ sa_add_rules(struct sa_ctx *sa_ctx, const struct ipsec_sa entries[], rc = create_ipsec_esp_flow(sa); if (rc != 0) RTE_LOG(ERR, IPSEC_ESP, - "create_ipsec_esp_flow() failed %s\n", - strerror(rc)); + "create_ipsec_esp_flow() failed\n"); } print_one_sa_rule(sa, inbound); }