[ovs-dev] [PATCH] oss-fuzz: Free error string in ofctl_parse_flow

Ben Pfaff blp at ovn.org
Fri Nov 2 18:32:35 UTC 2018


No problem.  I fixed that up and applied this to master.

On Thu, Nov 01, 2018 at 04:11:57PM -0700, Yifeng Sun wrote:
> Sorry, there is a typo in the message,
> the leaked free string => the leaked error string
> Ben, do you mind fixing it? Thanks.
> 
> Best,
> Yifeng
> 
> On Thu, Nov 1, 2018 at 11:40 AM Yifeng Sun <pkusunyifeng at gmail.com> wrote:
> 
> > This patch frees the leaked free string to stop oss-fuzz from
> > complaining.
> >
> > Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11161
> > Signed-off-by
> > <https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=11161Signed-off-by>:
> > Yifeng Sun <pkusunyifeng at gmail.com>
> > ---
> >  tests/oss-fuzz/ofctl_parse_target.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/oss-fuzz/ofctl_parse_target.c
> > b/tests/oss-fuzz/ofctl_parse_target.c
> > index 13d0899dbbae..8a906400a5cc 100644
> > --- a/tests/oss-fuzz/ofctl_parse_target.c
> > +++ b/tests/oss-fuzz/ofctl_parse_target.c
> > @@ -58,6 +58,7 @@ ofctl_parse_flow(const char *input, int command)
> >                                     command, &usable_protocols);
> >      if (error) {
> >          printf("Error encountered: %s\n", error);
> > +        free(error);
> >      } else {
> >          ofctl_parse_flows__(&fm, 1, usable_protocols);
> >      }
> > --
> > 2.7.4
> >
> >


More information about the dev mailing list