Fix leaks and an iterator error found by cppcheck.

git-svn-id: http://svn.drobilla.net/lad/trunk/ingen@2399 a436a847-0d15-0410-975c-d299462d15a1

2 files changed, 9 insertions, 1 deletions

diff --git a/src/engine/InputPort.cpp b/src/engine/InputPort.cpp
index 80ae836d..cc72a7c5 100644
--- a/src/engine/InputPort.cpp
+++ b/src/engine/InputPort.cpp
@@ -163,10 +163,16 @@ InputPort::remove_connection(const OutputPort* src_port)
 	ThreadManager::assert_thread(THREAD_PROCESS);
 
 	Connections::Node* connection = NULL;
-	for (Connections::iterator i = _connections.begin(); i != _connections.end(); ++i)
+	for (Connections::iterator i = _connections.begin(); i != _connections.end();) {
+		Connections::iterator next = i;
+		++next;
+
 		if ((*i)->src_port() == src_port)
 			connection = _connections.erase(i);
 
+		i = next;
+	}
+
 	if ( ! connection) {
 		error << "[InputPort::remove_connection] Connection not found!" << endl;
 		return NULL;
diff --git a/src/engine/PatchImpl.cpp b/src/engine/PatchImpl.cpp
index 96841b33..771c9d6d 100644
--- a/src/engine/PatchImpl.cpp
+++ b/src/engine/PatchImpl.cpp
@@ -377,6 +377,7 @@ PatchImpl::remove_port(const string& symbol)
 		if ((*i)->symbol() == symbol) {
 			ret = _input_ports.erase(i);
 			found = true;
+			break;
 		}
 	}
 
@@ -385,6 +386,7 @@ PatchImpl::remove_port(const string& symbol)
 		if ((*i)->symbol() == symbol) {
 			ret = _output_ports.erase(i);
 			found = true;
+			break;
 		}
 	}