PATRYK/Telodendria
1
0
Fork 0
forked from Telodendria/Telodendria
Code Pull requests Activity

Fix segfault in fclose() on Linux.

Browse source
This commit is contained in:
Jordan Bancino 2022-11-02 00:35:05 +00:00
parent 5af229a86e
commit 6992f36c86
3 changed files with 15 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
src/HttpServer.c
View file

@ -167,9 +167,7 @@ HttpServerContextFree(HttpServerContext * c)
HashMapFree(c->requestParams); HashMapFree(c->requestParams);
Free(c->requestPath); Free(c->requestPath);
#if 0
fclose(c->stream); fclose(c->stream);
#endif
Free(c); Free(c);
} }
@ -359,15 +357,15 @@ HttpServerCreate(unsigned short port, unsigned int nThreads, unsigned int maxCon
goto error; goto error;
} }
if (setsockopt(server->sd, SOL_SOCKET, SO_REUSEADDR, &ENABLE, sizeof(int)) < 0) if (setsockopt(server->sd, SOL_SOCKET, SO_REUSEADDR, &ENABLE, sizeof(int)) < 0)
{ {
goto error; goto error;
} }
if (setsockopt(server->sd, SOL_SOCKET, SO_REUSEPORT, &ENABLE, sizeof(int)) < 0) if (setsockopt(server->sd, SOL_SOCKET, SO_REUSEPORT, &ENABLE, sizeof(int)) < 0)
{ {
goto error; goto error;
} }
sa.sin_family = AF_INET; sa.sin_family = AF_INET;
sa.sin_port = htons(port); sa.sin_port = htons(port);
@ -596,6 +594,8 @@ HttpServerWorkerThread(void *args)
server->requestHandler(context, server->handlerArgs); server->requestHandler(context, server->handlerArgs);
HttpServerContextFree(context); HttpServerContextFree(context);
fp = NULL; /* The above call will close this
* FILE */
goto finish; goto finish;
internal_error: internal_error:
@ -610,7 +610,10 @@ bad_request:
finish: finish:
Free(line); Free(line);
fclose(fp); if (fp)
{
fclose(fp);
}
} }
return NULL; return NULL;

3
src/Matrix.c
View file

@ -144,9 +144,6 @@ MatrixHttpHandler(HttpServerContext * context, void *argp)
JsonFree(response); JsonFree(response);
finish: finish:
stream = HttpStream(context);
fclose(stream);
LogConfigUnindent(lc); LogConfigUnindent(lc);
} }

2
src/Telodendria.c
View file

@ -211,7 +211,7 @@ main(int argc, char **argv)
} }
else else
{ {
fclose(stdin); fclose(stdin);
#ifdef __OpenBSD__ #ifdef __OpenBSD__
if (unveil(configArg, "r") != 0) if (unveil(configArg, "r") != 0)
{ {