[ovs-git] Open vSwitch: vswitchd: Initialize cfg properly and check return values (citrix)

nicira-dev at nicira.com nicira-dev at nicira.com
Mon Aug 3 18:26:06 UTC 2009

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, citrix has been updated
       via  110d69af04a250cc2560644edd65a27158fa45e3 (commit)
       via  590ab3cc649ca3ecc56e6c5057e7e4e24874a625 (commit)
      from  85c74638ed99b77d6894385c3dce0175c4b4baa4 (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 110d69af04a250cc2560644edd65a27158fa45e3
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=110d69af04a250cc2560644edd65a27158fa45e3
Author: Justin Pettit <jpettit at nicira.com>
vswitchd: Initialize cfg properly and check return values
A previous checkin added the cfg_init() function, so we now call it.  We
also check the return value of the initial call to cfg_read(), since if
it fails, there's not much point in continuing.

commit 590ab3cc649ca3ecc56e6c5057e7e4e24874a625
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=590ab3cc649ca3ecc56e6c5057e7e4e24874a625
Author: Justin Pettit <jpettit at nicira.com>
cfg: Terminate cfg to prevent crashes
If cfg_* accessor calls were made before cfg_read() was called (or it
returned error), they could cause segfault.  This checkin terminates the
cfg structure in such a way that will prevent these run-time problems.

Bug #1693


Summary of changes:
 lib/cfg.c                |    7 +++++++
 lib/cfg.h                |    1 +
 utilities/ovs-cfg-mod.c  |    1 +
 vswitchd/ovs-brcompatd.c |    6 +++++-
 vswitchd/ovs-vswitchd.c  |    6 +++++-
 5 files changed, 19 insertions(+), 2 deletions(-)

Open vSwitch

More information about the git mailing list