[ovs-git] Open vSwitch: ovsdb-server: Add support for multiple databases. (master)

dev at openvswitch.org dev at openvswitch.org
Tue Sep 25 16:20:06 UTC 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, master has been updated
       via  b4e8d17057ff29183c026a7bbfc751520e1629f5 (commit)
       via  06834871348ac2c23418064a3e04d769ac5373ec (commit)
       via  c824c8a39854199a4221256281b1322cfae33469 (commit)
      from  5396b42ca2a14c760cb2d32e7ec59525866fddbb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit b4e8d17057ff29183c026a7bbfc751520e1629f5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=b4e8d17057ff29183c026a7bbfc751520e1629f5
Author: Ben Pfaff <blp at nicira.com>
		
ovsdb-server: Add support for multiple databases.
		
The OVSDB protocol has supported multiple databases for a long time, but
the ovsdb-server implementation only supported one database at a time.
This commit adds support for multiple databases.

Feature #12353.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>


commit 06834871348ac2c23418064a3e04d769ac5373ec
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=06834871348ac2c23418064a3e04d769ac5373ec
Author: Ben Pfaff <blp at nicira.com>
		
ovsdb-server: Add --enable-dummy option for unit tests.
		
This will allow an upcoming unit test to use the "time/warp" ovs-appctl
command with ovsdb-server.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>


commit c824c8a39854199a4221256281b1322cfae33469
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c824c8a39854199a4221256281b1322cfae33469
Author: Ben Pfaff <blp at nicira.com>
		
ovsdb-server: Fix null pointer deref when bool "is_connected" is empty.
		
The ovsdb-server supports obtaining its remote connection targets from a
database table and updating that table with connection status information.
One of the supported connection status columns is a boolean column named
"is_connected".  The code in ovsdb-server blindly assigned a bool into
this column without checking that it actually had space allocated for one.
This was and is fine with the ovs-vswitchd schema, which always has exactly
one value in this column.  However, if a database schema makes this column
optional, and there are actually no values in it, then this assignment
dereferences a null pointer.

This commit fixes the problem by allocating space for a bool if none has
yet been allocated.

Noticed while adding an extra test for the connection status feature.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 ovsdb/jsonrpc-server.c  |  115 +++++++++++-------
 ovsdb/jsonrpc-server.h  |    4 +-
 ovsdb/ovsdb-server.1.in |   32 +++---
 ovsdb/ovsdb-server.c    |  299 ++++++++++++++++++++++++++++++++--------------
 ovsdb/server.c          |   28 ++++-
 ovsdb/server.h          |   13 ++-
 ovsdb/trigger.c         |    9 +-
 ovsdb/trigger.h         |    8 +-
 tests/ovsdb-server.at   |   61 ++++++++---
 tests/test-ovsdb.c      |    8 +-
 utilities/ovs-ctl.8     |    6 +
 utilities/ovs-ctl.in    |   17 +++-
 12 files changed, 414 insertions(+), 186 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list