On Fri, May 18, 2018 at 10:44:04AM +1000, Dave Chinner wrote: > On Thu, May 17, 2018 at 12:27:00PM -0700, Luis R. Rodriguez wrote: > > +} > > + > > +static enum parse_line_type parse_get_line_type(const char *line, char *tag, > > + bool *value) > > +{ > > + int ret; > > + unsigned int uint_value; > > + > > + memset(tag, 0, 80); > > Why? This should be done in the caller, and if there's a fixed > buffer size, then please use a #define. I've modified to use 'm' specifier on scanf(). Using 'm' tells scanf() to allocate a buffer for us, this is easier as we then enable the parser to do as it sees fit so long as the patterns match. We just have to later free it after use. That removes the need to memset() and stick to a predefined buffer sizes for the tags (section or subopt name). Luis -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html