On 2/4/2020 2:39 PM, Iremonger, Bernard wrote: >> -----Original Message----- >> From: Ori Kam <or...@mellanox.com> >> Sent: Tuesday, February 4, 2020 1:40 PM >> To: Lu, Wenzhuo <wenzhuo...@intel.com>; Wu, Jingjing >> <jingjing...@intel.com>; Iremonger, Bernard >> <bernard.iremon...@intel.com> >> Cc: dev@dpdk.org; or...@mellanox.com; Yigit, Ferruh >> <ferruh.yi...@intel.com>; viachesl...@mellanox.com >> Subject: [PATCH v4] app/testpmd: fix copying the name of the dynflag >> >> When working with testpmd and setting the dynflag name, we copy the >> name given by the cmd to the dynflag name. >> >> The issue is that the size of the dynflag name is smaller then the string >> used >> by testpmd. >> >> This commit solves this issue by checking that the length of the requested >> flag name is not too long. >> >> Coverity issue: 353610 >> >> Fixes: b57b66a97ebf ("app/testpmd: support mbuf dynamic flag") >> >> Signed-off-by: Ori Kam <or...@mellanox.com> > > Acked-by: Bernard Iremonger <bernard.iremon...@intel.com> >
Applied to dpdk-next-net/master, thanks.