Subj : CVS commit src/sbbs3/websrvr.c To : All From : rswindell Date : Tue Mar 29 2005 01:50 am src/sbbs3 websrvr.c 1.297 1.298 Update of /cvsroot/sbbs/src/sbbs3 In directory cvs.synchro.net:/tmp/cvs-serv19488 Modified Files: websrvr.c Log Message: If strListInit() returns NULL (malloc failure), log error and break http_session_thread loop (should send a "system error" status to client). Also added comment (for Deuce) regarding possible memory leak. It's rather challenging to follow all the possible code paths in http_session_thread and make sure they all eventually call close_request() (where the memory allocated for a request is freed), but it seems there may be one condition where the memory isn't being freed. Found this as a result of a tracing the cause of a NULL-pointer dereference exception: session->req.headers was NULL in a call to parse_headers(). .