[dpdk-dev] ipc: fix duplicate string copy
Checks
Commit Message
Coverity issue: 272582
Fixes: 2147c0950580 ("ipc: clean up code")
Cc: jianfeng.tan@intel.com
Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
lib/librte_eal/common/eal_common_proc.c | 1 -
1 file changed, 1 deletion(-)
Comments
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Anatoly Burakov
> Sent: Thursday, May 3, 2018 11:35 AM
> To: dev@dpdk.org
> Cc: Tan, Jianfeng <jianfeng.tan@intel.com>
> Subject: [dpdk-dev] [PATCH] ipc: fix duplicate string copy
>
> Coverity issue: 272582
>
> Fixes: 2147c0950580 ("ipc: clean up code")
> Cc: jianfeng.tan@intel.com
>
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
> > Coverity issue: 272582
> >
> > Fixes: 2147c0950580 ("ipc: clean up code")
> > Cc: jianfeng.tan@intel.com
> >
> > Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
>
> Acked-by: Harry van Haaren <harry.van.haaren@intel.com>
Applied, thanks
@@ -870,7 +870,6 @@ mp_request_async(const char *dst, struct rte_mp_msg *req,
pending_req->type = REQUEST_TYPE_ASYNC;
strlcpy(pending_req->dst, dst, sizeof(pending_req->dst));
- strcpy(pending_req->dst, dst);
pending_req->request = req;
pending_req->reply = reply_msg;
pending_req->async.param = param;