diff options
author | Ben Pfaff <blp@ovn.org> | 2017-05-26 15:56:57 -0700 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2017-05-30 07:48:09 -0700 |
commit | 56f9a57a103ee862a62ad2f254e3bdf48b5162e6 (patch) | |
tree | 364a0164b573407f9332808a3e34ba92310dc195 /ovn | |
parent | 9bd881f77861a8110b40de81730f9045287a58a6 (diff) | |
download | openvswitch-56f9a57a103ee862a62ad2f254e3bdf48b5162e6.tar.gz |
ovn-northd: Fix uninitialized reference in build_static_route_flow().
It was apparently possible for lrp_addr_s to be uninitialized in a logical
router with no ports.
Found by Coverity.
Reported-at: https://scan3.coverity.com/reports.htm#v16889/p10449/fileInstanceId=14763080&defectInstanceId=4304818&mergedDefectId=180426
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Miguel Angel Ajo <majopela@redhat.com>
Diffstat (limited to 'ovn')
-rw-r--r-- | ovn/northd/ovn-northd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 83db75338..94bbe7c48 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -3856,7 +3856,7 @@ build_static_route_flow(struct hmap *lflows, struct ovn_datapath *od, const struct nbrec_logical_router_static_route *route) { ovs_be32 nexthop; - const char *lrp_addr_s; + const char *lrp_addr_s = NULL; unsigned int plen; bool is_ipv4; |