Sorry for the late reply, but I noticed the test folder is recently moved under app so you may need a rebase.
Please keep my ack after rebase: Acked-by: Yipeng Wang <yipeng1.wang at intel.com> >-----Original Message----- >From: Poornima, PallantlaX >Sent: Monday, March 25, 2019 7:43 AM >To: dev@dpdk.org >Cc: Pattan, Reshma <reshma.pat...@intel.com>; Wang, Yipeng1 ><yipeng1.w...@intel.com>; Gobriel, Sameh ><sameh.gobr...@intel.com>; Richardson, Bruce <bruce.richard...@intel.com>; De >Lara Guarch, Pablo ><pablo.de.lara.gua...@intel.com>; step...@networkplumber.org; sta...@dpdk.org >Subject: RE: [PATCH v2] hash: fix sprintf with snprintf > >Hi, > >>-----Original Message----- >>From: Poornima, PallantlaX >>Sent: Thursday, February 14, 2019 6:28 PM >>To: dev@dpdk.org >>Cc: Pattan, Reshma <reshma.pat...@intel.com>; Wang, Yipeng1 >><yipeng1.w...@intel.com>; Gobriel, Sameh <sameh.gobr...@intel.com>; >>Richardson, Bruce <bruce.richard...@intel.com>; De Lara Guarch, Pablo >><pablo.de.lara.gua...@intel.com>; step...@networkplumber.org; >>Poornima, PallantlaX <pallantlax.poorn...@intel.com>; sta...@dpdk.org >>Subject: [PATCH v2] hash: fix sprintf with snprintf >> >>sprintf function is not secure as it doesn't check the length of string. >>More secure function snprintf is used. >> >>Fixes: 473d1bebce ("hash: allow to store data in hash table") >>Cc: sta...@dpdk.org >> >>Signed-off-by: Pallantla Poornima <pallantlax.poorn...@intel.com> >>--- >>v2: Addressed review comment to correct the format specifier of >>hastest_key_lens. >>--- > > >Request for ACK please if there are no comments > >Regards, >Poornima