From d55e29d26792934dc4972209259f9081929a1789 Mon Sep 17 00:00:00 2001
From: erentar <erentar2002@gmail.com>
Date: Fri, 13 Jun 2025 15:01:50 +0200
Subject: [PATCH] fix segfault when allocating buffers bigger than stack size,
 use malloc instead

---
 fiche.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/fiche.c b/fiche.c
index 99d140d..f7685fa 100644
--- a/fiche.c
+++ b/fiche.c
@@ -561,7 +561,8 @@ static void *handle_connection(void *args) {
     }
 
     // Create a buffer
-    uint8_t buffer[c->settings->buffer_len];
+    // uint8_t buffer[c->settings->buffer_len];
+    uint8_t* buffer = malloc(c->settings->buffer_len);
     memset(buffer, 0, c->settings->buffer_len);
 
     const int r = recv(c->socket, buffer, sizeof(buffer), MSG_WAITALL);
@@ -574,6 +575,7 @@ static void *handle_connection(void *args) {
 
         // Cleanup
         free(c);
+        free(buffer);
         pthread_exit(NULL);
 
         return 0;
@@ -614,9 +616,10 @@ static void *handle_connection(void *args) {
             print_separator();
 
             // Cleanup
+            close(c->socket);
             free(c);
             free(slug);
-            close(c->socket);
+            free(buffer);
             pthread_exit(NULL);
             return NULL;
         }
@@ -634,6 +637,7 @@ static void *handle_connection(void *args) {
 
         // Cleanup
         free(c);
+        free(buffer);
         pthread_exit(NULL);
         return NULL;
     }
@@ -649,6 +653,7 @@ static void *handle_connection(void *args) {
         // Cleanup
         free(c);
         free(slug);
+        free(buffer);
         pthread_exit(NULL);
         return NULL;
     }
@@ -678,6 +683,7 @@ static void *handle_connection(void *args) {
     // Perform cleanup of values used in this thread
     free(slug);
     free(c);
+    free(buffer);
 
     pthread_exit(NULL);