daryaevd / osi_threads Goto Github PK

Потенциальный двойной вызов free

double free or corruption (out)
zsh: IOT instruction (core dumped)


(gdb) where
...
#9  0x000056334fd45a52 in handleRequestConnection (args=0x56334ff7ca70) at proxy.c:168
#10 0x00007ff0cd6199eb in start_thread (arg=<optimized out>) at pthread_create.c:444
#11 0x00007ff0cd69d7cc in clone3 () at ../sysdeps/unix/sysv/linux/x86_64/clone3.S:78

1.3

Освобождение памяти происходит не при всех вариантах/не во всех ветках исполнения программы

1.6

Нужно вынести библиотеку в отдельный файл и не смешивать ее с частным примером ее использования (main)

Кажется, exit - довольно странный способ сообщить об ошибке в библиотеке

usleep(1) - довольно маленький интервал, процессор будет слишком много раз вхолостую обращать внимание на процесс

2.2

if (pthread_mutex_unlock(&q->mutex)) {
    printf("pthread_mutex_unlock() error\n");
  }

общий код (типа такого) нужно вынести в отдельный метод