On Wed, Sep 11, 2013 at 5:19 PM, Alex Wang <al...@nicira.com> wrote:
> This commit fixes a place in bridge.c where smap_destroy() is not
> always called after smap_init().  Though there is no memory leak
> now, it is necessary to fix it and prevent memory leak in the
> future when smap_init() may be modified to allocate dynamic memory.
>
> Reported-by: Ansis Atteka <aatt...@nicira.com>
> Signed-off-by: Alex Wang <al...@nicira.com>
> ---
>  vswitchd/bridge.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Looks good to me.

We usually do not specify in subject line "bridge.c:...". Instead we
simply say "bridge:...". Can you fix this small thing before you push
this patch to git repository?
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to