[ovs-dev] [PATCH] ovn-controller: Fix memory leak in create_br_int().

Miguel Angel Ajo Pelayo majopela at redhat.com
Mon May 29 11:34:29 UTC 2017


Acked-By: Miguel Angel Ajo <majopela at redhat.com>

On Sat, May 27, 2017 at 1:17 AM, Ben Pfaff <blp at ovn.org> wrote:

> Found by Coverity.
>
> Reported-at: https://scan3.coverity.com/reports.htm#v16889/p10449/
> fileInstanceId=14763066&defectInstanceId=4305324&mergedDefectId=180404&
> fileStart=251&fileEnd=500
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
>  ovn/controller/ovn-controller.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-
> controller.c
> index f22551d6dcf3..1ff1b5b738a9 100644
> --- a/ovn/controller/ovn-controller.c
> +++ b/ovn/controller/ovn-controller.c
> @@ -238,6 +238,7 @@ create_br_int(struct controller_ctx *ctx,
>      bridges[cfg->n_bridges] = bridge;
>      ovsrec_open_vswitch_verify_bridges(cfg);
>      ovsrec_open_vswitch_set_bridges(cfg, bridges, cfg->n_bridges + 1);
> +    free(bridges);
>
>      return bridge;
>  }
> --
> 2.10.2
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list