From 65075b99a72b88e4c4d6f991496a9e824aa87479 Mon Sep 17 00:00:00 2001
From: Simon Horman <simon.horman@netronome.com>
Date: Wed, 12 Nov 2014 15:24:08 +0900
Subject: [PATCH 1/1] ofp-util: Do not allow buckets for OFPGC15_REMOVE_BUCKET.

ONF-JIRA: EXT-350
Signed-off-by: Simon Horman <simon.horman@netronome.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
---
 lib/ofp-util.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 496dc404e..f1431d54f 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -7973,6 +7973,21 @@ ofputil_decode_group_mod(const struct ofp_header *oh,
         OVS_NOT_REACHED();
     }
 
+    switch (gm->command) {
+    case OFPGC11_ADD:
+    case OFPGC11_MODIFY:
+    case OFPGC11_DELETE:
+    case OFPGC15_INSERT_BUCKET:
+        break;
+    case OFPGC15_REMOVE_BUCKET:
+        if (!list_is_empty(&gm->buckets)) {
+            return OFPERR_OFPGMFC_BAD_BUCKET;
+        }
+        break;
+    default:
+        OVS_NOT_REACHED();
+    }
+
     LIST_FOR_EACH (bucket, list_node, &gm->buckets) {
         switch (gm->type) {
         case OFPGT11_ALL:
-- 
2.20.1