[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