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 <b...@nicira.com> --- ofproto/ofproto.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index 3a60328..1a1113f 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); } -- 1.7.10.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev