[ovs-dev] [coverity 04/14] ofproto: Free "controller_name" in ofconn_run().

Justin Pettit jpettit at nicira.com
Tue Feb 22 01:44:47 UTC 2011


Coverity #10729
---
 ofproto/ofproto.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index e4c3528..3324b83 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -1840,6 +1840,7 @@ ofconn_run(struct ofconn *ofconn)
                 char *ofconn_name = ofconn_make_name(p, controller_name);
                 rconn_connect(ofconn->rconn, controller_name, ofconn_name);
                 free(ofconn_name);
+                free(controller_name);
             } else {
                 rconn_disconnect(ofconn->rconn);
             }
-- 
1.7.1





More information about the dev mailing list