projects
/
openvswitch
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
b16fdaf
)
vconn: Fix byte-swapping of message type when validating OpenFlow actions.
author
Jean Tourrihles
<jt@hpl.hp.com>
Fri, 11 Sep 2009 22:19:15 +0000
(15:19 -0700)
committer
Ben Pfaff
<blp@nicira.com>
Fri, 11 Sep 2009 22:19:15 +0000
(15:19 -0700)
It seems really strange that this one slipped through. Perhaps this
means that we have never tested with any action other than OFPAT_OUTPUT
(which has value 0 and thus is not affected by byte-swapping).
lib/vconn.c
patch
|
blob
|
history
diff --git
a/lib/vconn.c
b/lib/vconn.c
index b5e110204eef1d64d925bb79731237601038f759..12680c5ba54d9f9a7ecd82dd01dbf01a372402b4 100644
(file)
--- a/
lib/vconn.c
+++ b/
lib/vconn.c
@@
-1247,7
+1247,7
@@
check_action(const union ofp_action *a, unsigned int len, int max_ports)
{
int error;
- switch (
a->type
) {
+ switch (
ntohs(a->type)
) {
case OFPAT_OUTPUT:
error = check_action_port(ntohs(a->output.port), max_ports);
if (error) {