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 f58368a8f56e5ae40cb5fcb1c1036315c0b17e5b..c3079b07c75f78573c80d2e8bbb2219024fef443 100644 (file)
--- 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