git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 93e039f) | patch
rconn: Clarify rconn_run_wait().
authorBen Pfaff <blp@nicira.com>
Wed, 23 Feb 2011 20:35:32 +0000 (12:35 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 23 Feb 2011 23:14:42 +0000 (15:14 -0800)
commit82c2b79de650665f18d847c1771d28430486225e
tree236384b52cf2138328d76948e1a81af149a76a10tree | snapshot
parent93e039fb5e7ae77a556e1d144ca069a86555598bcommit | diff
rconn: Clarify rconn_run_wait().

At first glance the vconn_wait() call looks risky because this function
checked whether rc->vconn is nonnull at the top.  In fact it's OK because
rc->state will be S_ACTIVE or S_IDLE only if rc->vconn is nonnull, but
there's no harm in putting that check inside the block that only runs if
rc->vconn is nonnull.

Coverity #10714.
lib/rconn.c diff | blob | history
sliver-openvswitch