On Fri, Jun 26, 2015 at 09:19:21AM +0200, Florian Westphal wrote:
> pfeiffer.szilard@xxxxxxxxxx <pfeiffer.szilard@xxxxxxxxxx> wrote:
> > From: Szilárd Pfeiffer <pfeiffer.szilard@xxxxxxxxxx>
> >
> > Extensions register protocols by lowercase protocol name, but value of
> > proto command line option may be uppercase. Extension related options
> > cannot be used when protocol name comparision fails.
> >
> > Signed-off-by: Szilárd Pfeiffer <pfeiffer.szilard@xxxxxxxxxx>
> > ---
> > src/conntrack.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/src/conntrack.c b/src/conntrack.c
> > index f7d37fb..b1a2589 100644
> > --- a/src/conntrack.c
> > +++ b/src/conntrack.c
> > @@ -555,6 +555,12 @@ static struct ctproto_handler *findproto(char *name, int *pnum)
> > /* using the protocol name for an unsupported protocol? */
> > if ((pent = getprotobyname(name))) {
> > *pnum = pent->p_proto;
>
> Why not use strcasecmp to check the proto_list?
Please, address Florian's feedback and resubmit, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
