On Mon, Dec 09, 2013 at 09:30:21PM -0800, Alan Coopersmith wrote: > Fixes cppcheck error: > [readfile.c:108]: (error) Common realloc mistake: 'cp' nulled > but not freed upon failure > > Signed-off-by: Alan Coopersmith <[email protected]>
Reviewed-by: Peter Hutterer <[email protected]> Cheers, Peter > --- > readfile.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/readfile.c b/readfile.c > index 9729854..122020f 100644 > --- a/readfile.c > +++ b/readfile.c > @@ -104,9 +104,11 @@ get_data_from_stdin (int *len_return) > count += n; > /* Here count <= allocated. Prepare for next round. */ > if (count + BUFSIZ > allocated) { > + char *oldp = cp; > allocated = 2 * allocated; > cp = realloc (cp, allocated + 1); > if (!cp) { > + free(oldp); > fprintf(stderr, "cannot get memory for message file\n"); > return NULL; > } > -- > 1.7.9.2 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel > _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
