[ovs-dev] [PATCH] ofp-flow: Fix uninitialized data decoding OF1.5 flow stats.
Ben Pfaff
blp at ovn.org
Wed May 23 20:51:59 UTC 2018
Reported-by: Paul Greenberg
Reported-at: https://github.com/openvswitch/ovs-issues/issues/149
Fixes: c7b02b800615 ("Add support for OpenFlow 1.5 statistics (OXS).")
Signed-off-by: Ben Pfaff <blp at ovn.org>
---
AUTHORS.rst | 1 +
lib/ofp-flow.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/AUTHORS.rst b/AUTHORS.rst
index c93d5be5f72c..9c81ad8b5d2d 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -533,6 +533,7 @@ Niklas Andersson nandersson at nicira.com
Pankaj Thakkar thakkar at nicira.com
Pasi Kärkkäinen pasik at iki.fi
Patrik Andersson R patrik.r.andersson at ericsson.com
+Paul Greenberg
Paulo Cravero pcravero at as2594.net
Pawan Shukla shuklap at vmware.com
Periyasamy Palanisamy periyasamy.palanisamy at ericsson.com
diff --git a/lib/ofp-flow.c b/lib/ofp-flow.c
index 9fbe8c2e5cb2..4b893ce959d4 100644
--- a/lib/ofp-flow.c
+++ b/lib/ofp-flow.c
@@ -955,6 +955,7 @@ ofputil_decode_flow_stats_reply(struct ofputil_flow_stats *fs,
fs->packet_count = oxs.packet_count;
fs->byte_count = oxs.byte_count;
fs->idle_age = oxs.idle_age == UINT32_MAX ? -1 : oxs.idle_age;
+ fs->hard_age = -1;
instructions_len = length - sizeof *ofd - padded_match_len - stat_len;
} else if (raw == OFPRAW_OFPST11_FLOW_REPLY
--
2.16.1
More information about the dev
mailing list