Fix some memory leak issues.

Free memory in some places.
crapulent · Sep 6, 2016 · 506a62b · 506a62b
1 parent 4b500ee
commit 506a62b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/openmcu-ru/reg_sip.cxx b/openmcu-ru/reg_sip.cxx
@@ -269,9 +269,15 @@ int Registrar::OnReceivedSipInvite(const msg_t *msg)
     if(raccount_out) raccount_out->Unlock();
     if(rconn) rconn->Unlock();
     if(response_code == 0)
+    {
+		msg_destroy(msg_reply);
       return 0;
+	}
     else if(response_code == -1)
+    {
       sep->CreateIncomingConnection(msg); // MCU call
+		msg_destroy(msg_reply);
+	}
     else
       sep->SipReqReply(msg, msg_reply, response_code);
     return 1;

diff --git a/openmcu-ru/sockets.cxx b/openmcu-ru/sockets.cxx
@@ -119,6 +119,7 @@ BOOL MCUSocket::GetFromIP(PString & ip, const PString & host, const PString & po
     PTRACE(1, "MCUSocket error " << errno << " " << strerror(errno));
     return FALSE;
   }
+  freeaddrinfo(res);
 
   sockaddr_in name;
   socklen_t namelen = sizeof(name);

diff --git a/openmcu-ru/utils_av.cxx b/openmcu-ru/utils_av.cxx
@@ -476,14 +476,14 @@ BOOL MCU_AVDecodeFrameFromFile(PString & filename, void *dst, int & dst_size, in
   }
 
   end:
-    if(fmt_ctx)
-      avformat_close_input(&fmt_ctx);
     if(context)
     {
       avcodecMutex.Wait();
       avcodec_close(context);
       avcodecMutex.Signal();
     }
+    if(fmt_ctx)
+      avformat_close_input(&fmt_ctx);
     if(frame)
       AVFrameFree(&frame);