Thanks for the tip. I'll resend a v2, ;D
On Thu, Sep 12, 2013 at 9:52 AM, Ansis Atteka <aatt...@nicira.com> wrote: > 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
