Acked-by: Dennis Flynn <[email protected]>

On Thu, Apr 16, 2015 at 5:46 PM, Ben Pfaff <[email protected]> wrote:

> Found by LLVM scan-build.
>
> Reported-by: Kevin Lo <[email protected]>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/ovs-lldp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/ovs-lldp.c b/lib/ovs-lldp.c
> index 9ee639e..bf1b61d 100644
> --- a/lib/ovs-lldp.c
> +++ b/lib/ovs-lldp.c
> @@ -619,7 +619,6 @@ aa_mapping_unregister(void *aux)
>              }
>
>              hmap_remove(&lldp->mappings_by_aux, &m->hmap_node_aux);
> -            free(m);
>
>              /* Remove from all the lldp instances */
>              LIST_FOR_EACH (hw, h_entries, &lldp->lldpd->g_hardware) {
> @@ -629,6 +628,7 @@ aa_mapping_unregister(void *aux)
>
>                  aa_mapping_unregister_mapping(lldp, hw, m);
>              }
> +            free(m);
>
>              /* Remove from the all_mappings */
>              HMAP_FOR_EACH (m, hmap_node_isid, all_mappings) {
> --
> 2.1.3
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to