On Fri, Jan 09, 2009 at 02:00:01PM -0500, Jeff Moyer wrote:
> Valerie Aurora Henson <vaur...@redhat.com> writes:
> 
> > Signed-off-by: Valerie Aurora Henson <vaur...@redhat.com>
> 
> > @@ -1524,18 +1525,12 @@ int lookup_read_master(struct master *master, 
> > time_t age, void *context)
> >                     goto next;
> >             }
> >  
> > -           blen = strlen(*keyValue) + 1 + strlen(*values) + 2;
> > -           if (blen > PARSE_MAX_BUF) {
> > +           if (snprintf(parse_buf, sizeof(parse_buf), "%s %s",
> > +                        *keyValue, *values) > sizeof(parse_buf)) {
> 
> I think this should be >=.

Yes, they all should be.  I'll regenerate.  Thanks,

-VAL

_______________________________________________
autofs mailing list
autofs@linux.kernel.org
http://linux.kernel.org/mailman/listinfo/autofs

Reply via email to