Acked-by: Jarno Rajahalme <jrajaha...@nicira.com>

On Dec 10, 2013, at 11:19 PM, Ben Pfaff <b...@nicira.com> wrote:

> 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

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to