Re: [PATCH] tools/xenstored: Wire properly the command line option -M/--path-max

[Julien Grall <julien@xxxxxxx>]

Hi Juergen,

On 21/04/2021 16:07, Juergen Gross wrote:
> On 21.04.21 16:02, Julien Grall wrote:
> > From: Julien Grall <jgrall@xxxxxxxxxx>
> >
> > The command line option -M/--path-max was meant to be added by
> > commit 924bf8c793cb "tools/xenstore: rework path length check" but this
> > wasn't wired through properly.
> >
> > Fix it by adding the missing "case 'M':".
> >
> > Fixes: 924bf8c793cb ("tools/xenstore: rework path length check")
> > Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
Thanks! I have committed it.

> I wonder why no compiler cared about the obviously dead coding.
Coverity actually spotted it (see CID-1470790) back in December 2020. 
Although, I wasn't aware of it because I just check coverity now.
Cheers,

--
Julien Grall