[ovs-git] [openvswitch/ovs] 71bf92: ovn-controller: Fix invalid br_int_name handling.

GitHub noreply at github.com
Fri May 1 23:03:26 UTC 2015


  Branch: refs/heads/ovn
  Home:   https://github.com/openvswitch/ovs
  Commit: 71bf929e647acbe4cf186ebf72ca10f10317817c
      https://github.com/openvswitch/ovs/commit/71bf929e647acbe4cf186ebf72ca10f10317817c
  Author: Russell Bryant <rbryant at redhat.com>
  Date:   2015-05-01 (Fri, 01 May 2015)

  Changed paths:
    M ovn/controller/ovn-controller.c
    M ovn/controller/ovn-controller.h

  Log Message:
  -----------
  ovn-controller: Fix invalid br_int_name handling.

While doing some testing, I noticed the following error message:

  Integration bridge '<garbage>' dissapeared

The reason is that the code kept around the value of the "ovn-bridge"
configuration without copying it.  The result was pointing to bogus
memory.  You would only see this if you set "ovn_bridge".  If you
relied on the default, the bug would not occur.

Signed-off-by: Russell Bryant <rbryant at redhat.com>
Acked-by: Kyle Mestery <mestery at mestery.com>
Signed-off-by: Justin Pettit <jpettit at nicira.com>


  Commit: 07c747d036eefc54aff602dd638485adbb2099cd
      https://github.com/openvswitch/ovs/commit/07c747d036eefc54aff602dd638485adbb2099cd
  Author: Russell Bryant <rbryant at redhat.com>
  Date:   2015-05-01 (Fri, 01 May 2015)

  Changed paths:
    M ovn/controller/ovn-controller.c

  Log Message:
  -----------
  ovn-controller: Free some more memory before exit.

Free a few pointers to malloc'd memory before exit.

Signed-off-by: Russell Bryant <rbryant at redhat.com>
Acked-by: Kyle Mestery <mestery at mestery.com>
Signed-off-by: Justin Pettit <jpettit at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/e4901fe06724...07c747d036ee


More information about the git mailing list