> -----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>