From 89843f1eca65678cf7e751f7c2a636ee9d1d8b41 Mon Sep 17 00:00:00 2001
From: Donald Buczek <buczek@molgen.mpg.de>
Date: Thu, 20 May 2021 09:49:51 +0200
Subject: [PATCH] mxshadowsrc: Remove DEBUG_MAX_CONNECTS

Remove debug termination logic, which will be reimplemented in a
different way when we prefork.
---
 mxshadowsrv.c | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/mxshadowsrv.c b/mxshadowsrv.c
index a867fa5..5eb2ed1 100644
--- a/mxshadowsrv.c
+++ b/mxshadowsrv.c
@@ -7,10 +7,6 @@
 #include <string.h>
 #include <getopt.h>
 #include <pthread.h>
-#include <semaphore.h>
-#ifdef DEBUG_MAX_CONNECTS
-#include <sys/wait.h>
-#endif
 
 #include "common.h"
 
@@ -76,9 +72,6 @@ static SSL_CTX *ssl_ctx;
 static sem_t free_worker;
 static int listen_socket;
 static char *filename;
-#ifdef DEBUG_MAX_CONNECTS
-static int debug_remaining_connects = DEBUG_MAX_CONNECTS;
-#endif
 
 static void validate_shadow(char **shadow_buf, char *filename, struct stat *statbuf) {
     int status = pthread_mutex_lock(&shadow_mutex);
@@ -271,10 +264,6 @@ int main(int argc, char **argv) {
     if (status) { errno = status; perror("sem_init"); exit(1); }
 
     while (1) {
-#ifdef DEBUG_MAX_CONNECTS
-        if (debug_remaining_connects-- == 0)
-            break;
-#endif
         int _cleanup_(free_fd) socket = accept4(listen_socket, NULL, NULL, SOCK_NONBLOCK);
         if (socket == -1 ) { perror("accept"); exit(1); }
         status = sem_wait(&free_worker);
@@ -289,16 +278,5 @@ int main(int argc, char **argv) {
         socket = -1;
         status = pthread_detach(thread);
         if (status != 0) { errno = status; perror("pthread_detach"); exit(1); }
-
-   }
-#ifdef DEBUG_MAX_CONNECTS
-    for (int i=0; i<MAX_THREADS; i++) {
-        status = sem_wait(&free_worker);
-        if (status == -1) { perror("sem_wait"); exit(1); }
     }
-    /* Valgrinds checks on exit() race with pthread_exit() and Valgrind might report
-     * a threads stack as lost memory or a Helgrind detected data race. Give threads
-     * a little time to exit after they posted the semaphore */
-    sleep(1);
-#endif
 }