28/04/2017 11:13, Olivier Matz:
> 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.
> > 
> > 
> > > 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>

Applied, thanks

Reply via email to