Re: [PATCH] Ignore whitespace at the beginning of option tags

[Malahal Naineni <malahal@us.ibm.com>]

Sorry Steve, Ben is right. It is based on
"git://git.linux-nfs.org/projects/steved/libnfsidmap.git"

Do I need to re-clone my repo with something else?

Regards, Malahal.

Benjamin Coddington [bcodding@redhat.com] wrote:
> On Tue, 2 Feb 2016, Steve Dickson wrote:
> >
> > On 01/28/2016 05:38 PM, Malahal Naineni wrote:
> > > The following should work now:
> > >
> > > [General]
> > > 	Domain = local.domain.edu
> > >
> > > Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
> > > ---
> > >  cfg.c | 9 ++++++++-
> > Ok.. I give... :-) What is this patch relative to? There is
> > a conf_parse_line() in support/nfs/conffile.c but there
> > is no cfg.c file in nfs-utils... and this patch does
> > not apply...
> 
> This is for libnfsidap..  cc-ing Bruce.
> 
> Ben
> 
> > >  1 file changed, 8 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/cfg.c b/cfg.c
> > > index c615d24..e0ab839 100644
> > > --- a/cfg.c
> > > +++ b/cfg.c
> > > @@ -210,6 +210,7 @@ static void
> > >  conf_parse_line (int trans, char *line, size_t sz)
> > >  {
> > >    char *val;
> > > +  char *tag;
> > >    size_t i;
> > >    int j;
> > >    static char *section = 0;
> > > @@ -262,8 +263,14 @@ conf_parse_line (int trans, char *line, size_t sz)
> > >  	/* Skip trailing whitespace, if any */
> > >  	for (j = sz - (val - line) - 1; j > 0 && isspace (val[j]); j--)
> > >  	  val[j] = '\0';
> > > +
> > > +	/* Skip beginning white space */
> > > +	tag = line;
> > > +	while (isspace(*tag))
> > > +		tag++;
> > > +
> > >  	/* XXX Perhaps should we not ignore errors?  */
> > > -	conf_set (trans, section, line, val, 0, 0);
> > > +	conf_set (trans, section, tag, val, 0, 0);
> > >  	return;
> > >        }
> > >
> > >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>