Fixed build

src/merge_mining_client_tari.cpp

@@ -212,12 +212,11 @@ void MergeMiningClientTari::on_external_block(const PoolBlock& block)
 
 				if (std::find(a, b, *data.u64()) == b) {
 					LOGINFO(4, "External aux job solution found, but it's for another miner");
+					return;
 				}
 				else {
 					LOGINFO(4, "External aux job solution found, but it's stale");
 				}
-
-				return;
 			}
 
 			if (!diff.check_pow(block.m_powHash)) {

src/util.h

@@ -69,7 +69,16 @@ template<typename T>
 struct ScopeGuard : public nocopy_nomove
 {
 	explicit FORCEINLINE ScopeGuard(T&& handler) : m_handler(std::move(handler)) {}
-	FORCEINLINE ~ScopeGuard() { m_handler(); }
+
+	FORCEINLINE ~ScopeGuard()
+	{
+		// Because the handler can throw an exception, and we don't want exceptions in a destructor
+		try {
+			m_handler();
+		}
+		catch(...) {
+		}
+	}
 
 	T m_handler;
 };

src/zmq_reader.cpp

@@ -171,12 +171,7 @@ void ZMQReader::run()
 		}
 
 		ON_SCOPE_LEAVE([this]() {
-			try {
 			m_monitor->abort();
-			}
-			catch(...) {
-				LOGERR(1, "m_monitor->abort() failed");
-			}
 			uv_thread_join(&m_monitorThread);
 		});