Message ID | 1367886028-23855-1-git-send-email-colin.king@canonical.com |
---|---|
State | Accepted |
Headers | show |
On 05/07/2013 08:20 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > Coverity CID #997339, Resource leak > > Fix memory leak, free line when we return. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > src/utilities/kernelscan.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/utilities/kernelscan.c b/src/utilities/kernelscan.c > index 95dc8a2..9acaa4e 100644 > --- a/src/utilities/kernelscan.c > +++ b/src/utilities/kernelscan.c > @@ -777,6 +777,7 @@ static int parse_kernel_message(parser *p, token *t) > for (;;) { > ret = get_token(p, t); > if (ret == EOF) { > + free(line); > return EOF; > } > > Acked-by: Alex Hung <alex.hung@canonical.com>
On 05/07/2013 08:20 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > Coverity CID #997339, Resource leak > > Fix memory leak, free line when we return. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > src/utilities/kernelscan.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/utilities/kernelscan.c b/src/utilities/kernelscan.c > index 95dc8a2..9acaa4e 100644 > --- a/src/utilities/kernelscan.c > +++ b/src/utilities/kernelscan.c > @@ -777,6 +777,7 @@ static int parse_kernel_message(parser *p, token *t) > for (;;) { > ret = get_token(p, t); > if (ret == EOF) { > + free(line); > return EOF; > } > > Acked-by: Ivan Hu <ivan.hu@canonical.com>
diff --git a/src/utilities/kernelscan.c b/src/utilities/kernelscan.c index 95dc8a2..9acaa4e 100644 --- a/src/utilities/kernelscan.c +++ b/src/utilities/kernelscan.c @@ -777,6 +777,7 @@ static int parse_kernel_message(parser *p, token *t) for (;;) { ret = get_token(p, t); if (ret == EOF) { + free(line); return EOF; }