On Wed, Aug 26, 2015 at 5:16 PM, Mike Frysinger <vapier@xxxxxxxxxx> wrote: > On 26 Aug 2015 15:49, Kees Cook wrote: >> --- a/libmount/src/mountP.h >> +++ b/libmount/src/mountP.h >> @@ -212,6 +212,8 @@ struct libmnt_table { >> int comms; /* enable/disable comment parsing */ >> char *comm_intro; /* First comment in file */ >> char *comm_tail; /* Last comment in file */ >> + char *buf; /* Memory for reading lines */ >> + size_t buf_size; /* Size of line-reading buffer */ > > just to be clear, this structure is zero-allocated right ? getline > needs the first call to have buf==NULL in order for everything to > fall into place. It is, yes. See mnt_new_table()'s calloc usage. -Kees -- Kees Cook Chrome OS Security -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html