Hi Wenzhuo, On Fri, 28 Apr 2017 08:38:21 +0000, "Lu, Wenzhuo" <wenzhuo...@intel.com> wrote: > Hi Oilvier, > As we discussed before, I send this patch. Would you like to take a look at > it? Better fix it before release 17.05, thanks. > > > > -----Original Message----- > > From: Lu, Wenzhuo > > Sent: Tuesday, April 25, 2017 11:11 AM > > To: dev@dpdk.org > > Cc: olivier.m...@6wind.com; Lu, Wenzhuo; sta...@dpdk.org > > Subject: [PATCH] lib/librte_cmdline: fix CLI parsing issue > > > > When parsing a CLI, all the CLI instances are checked one by one. Even if an > > instance already matches the CLI, the parsing will not stop for ambiguous > > check. > > The problem is that the following check may change the parsing result of the > > previous one even if the following instance doesn't match. > > > > Use a temporary validate for the parsing result when trying to match an > > instance and only store the result when it matches, so the previous result > > has no chance to be changed. > > > > Fixes: af75078fece3 ("first public release") > > CC: sta...@dpdk.org > > > > Signed-off-by: Wenzhuo Lu <wenzhuo...@intel.com>
Sorry for the delay, Acked-by: Olivier Matz <olivier.m...@6wind.com>