From: Ben Pfaff <blp@nicira.com>
Date: Thu, 12 Mar 2009 21:45:36 +0000 (-0700)
Subject: secchan: Treat invalid table IDs in stats requests as requests for no flows.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=408cae9478ac7463772760257a77a173982eed00;p=openvswitch

secchan: Treat invalid table IDs in stats requests as requests for no flows.

Before, we treated invalid table IDs as requests for any flows in the table
at all, but it makes more sense to treat them as requests for no flows at
all.  (The value 0xff is explicitly defined by OpenFlow to mean "all
tables".)
---

diff --git a/secchan/ofproto.c b/secchan/ofproto.c
index f58368a8..c3079b07 100644
--- a/secchan/ofproto.c
+++ b/secchan/ofproto.c
@@ -1998,7 +1998,8 @@ table_id_to_include(uint8_t table_id)
 {
     return (table_id == TABLEID_HASH ? CLS_INC_EXACT
             : table_id == TABLEID_CLASSIFIER ? CLS_INC_WILD
-            : CLS_INC_ALL);
+            : table_id == 0xff ? CLS_INC_ALL
+            : 0);
 }
 
 static int