From: Ben Pfaff <blp@nicira.com>
Date: Mon, 9 Dec 2013 23:38:25 +0000 (-0800)
Subject: ofproto: Make ofproto_enumerate_types() match its comment.
X-Git-Tag: sliver-openvswitch-2.1.90-1~10^2~220
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;ds=inline;h=c799c30657373c68b243bdb2576bf1883bcfce4a;p=sliver-openvswitch.git

ofproto: Make ofproto_enumerate_types() match its comment.

The comment says that it clears the passed-in sset, but it didn't.  The bug
didn't actually affect any of the existing callers, which all passed in an
empty sset.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jarno Rajahalme <jrajahalme@nicira.com>
---

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 424d2ced2..e41d72b62 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -432,6 +432,7 @@ ofproto_enumerate_types(struct sset *types)
 {
     size_t i;
 
+    sset_clear(types);
     for (i = 0; i < n_ofproto_classes; i++) {
         ofproto_classes[i]->enumerate_types(types);
     }