[ovs-dev] [bugfixes 5/6] ofproto: Fix memory leak in add_flow().

[Ethan Jackson]

Acked-by: Ethan Jackson <ethan at nicira.com>


On Wed, Sep 4, 2013 at 12:39 PM, Ben Pfaff <blp at nicira.com> wrote:
> Found by inspection.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  ofproto/ofproto.c |    1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 21155dc..97f26ae 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -3477,6 +3477,7 @@ add_flow(struct ofproto *ofproto, struct ofconn *ofconn,
>      error = ofproto_check_ofpacts(ofproto, fm->ofpacts, fm->ofpacts_len,
>                                    &fm->match.flow, table_id);
>      if (error) {
> +        cls_rule_destroy(&cr);
>          return error;
>      }
>
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev