mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-09-03 07:45:30 +00:00
Code Issues Releases Wiki Activity

ofp-util: Reject bad group type and command with error instead of abort.

Browse Source 
Reported-by: Manpreet Singh <er.manpreet25@gmail.com>
Reported-at: http://openvswitch.org/pipermail/discuss/2015-October/019048.html
Signed-off-by: Ben Pfaff <blp@ovn.org>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
This commit is contained in:
Ben Pfaff
2015-10-12 10:10:21 -07:00
parent 71300040d1
commit 09d4b95163
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
AUTHORS
View File

@@ -318,6 +318,7 @@ Len Gao leng@vmware.com
Logan Rosen logatronico@gmail.com Logan Rosen logatronico@gmail.com
Luca Falavigna dktrkranz@debian.org Luca Falavigna dktrkranz@debian.org
Luiz Henrique Ozaki luiz.ozaki@gmail.com Luiz Henrique Ozaki luiz.ozaki@gmail.com
Manpreet Singh er.manpreet25@gmail.com
Marco d'Itri md@Linux.IT Marco d'Itri md@Linux.IT
Martin Vizvary vizvary@ics.muni.cz Martin Vizvary vizvary@ics.muni.cz
Marvin Pascual marvin@pascual.com.ph Marvin Pascual marvin@pascual.com.ph

4
lib/ofp-util.c
View File

@@ -8679,7 +8679,7 @@ ofputil_decode_group_mod(const struct ofp_header *oh,
case OFPGT11_FF: case OFPGT11_FF:
break; break;
default: default:
OVS_NOT_REACHED(); return OFPERR_OFPGMFC_BAD_TYPE;
} }
switch (gm->command) { switch (gm->command) {
@@ -8694,7 +8694,7 @@ ofputil_decode_group_mod(const struct ofp_header *oh,
} }
break; break;
default: default:
OVS_NOT_REACHED(); return OFPERR_OFPGMFC_BAD_COMMAND;
} }
LIST_FOR_EACH (bucket, list_node, &gm->buckets) { LIST_FOR_EACH (bucket, list_node, &gm->buckets) {