From 594e9de349783ee869b959a68d444934e16ce487 Mon Sep 17 00:00:00 2001
From: "Bradford D. Boyle" <bradford.d.boyle@gmail.com>
Date: Sun, 10 Jun 2018 22:28:22 -0700
Subject: [PATCH] Change return type of `remove_thread()` to void

Flowing off the end of a function is undefined behavior. On Fedora 28,
the `nmux` program would segfault when this function returned.
---
 tsmpool.cpp | 2 +-
 tsmpool.h   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tsmpool.cpp b/tsmpool.cpp
index d70e8bae..7c8f88f2 100644
--- a/tsmpool.cpp
+++ b/tsmpool.cpp
@@ -42,7 +42,7 @@ tsmthread_t* tsmpool::register_thread()
 	return thread;
 }
 
-int tsmpool::remove_thread(tsmthread_t* thread)
+void tsmpool::remove_thread(tsmthread_t* thread)
 {
 	pthread_mutex_lock(&this->mutex);
 	for(int i=0;i<threads.size();i++)
diff --git a/tsmpool.h b/tsmpool.h
index d24cc4ef..96a289bf 100644
--- a/tsmpool.h
+++ b/tsmpool.h
@@ -36,7 +36,7 @@ class tsmpool
 	tsmpool(size_t size, int num);
 	void* get_write_buffer();
 	tsmthread_t* register_thread();
-	int remove_thread(tsmthread_t* thread);
+	void remove_thread(tsmthread_t* thread);
 	void* get_read_buffer(tsmthread_t* thread);
 	int index_next(int index) { return (index+1==num)?0:index+1; }
 	int index_before(int index) { return (index-1<0)?num-1:index-1; }