A BUGNOTE has been added to this bug. ====================================================================== http://dbmail.org/mantis/bug_view_advanced_page.php?bug_id=0000087 ====================================================================== Reported By: ljackson Assigned To: ilja ====================================================================== Project: DBMail Bug ID: 87 Category: Database layer Reproducibility: always Severity: major Priority: normal Status: confirmed ====================================================================== Date Submitted: 13-Sep-04 23:35 CEST Last Modified: 15-Sep-04 15:03 CEST ====================================================================== Summary: Possible memory leak in session and db code for pop3d Description: I have been able to reproduce in both cvs from 080304 and 2.0rc8 an issue where normal connects & lists to the pop3d leaks memory, I have traced it to session code not being freed in the connection cleanup code. ======================================================================
---------------------------------------------------------------------- ilja - 15-Sep-04 12:20 CEST ---------------------------------------------------------------------- I'm taking this patch one step at a time. first: I've added db_session_cleanup() to pop3.c. I've put it in a different place than in the patch. It should clean up the session after errors too. So, it's added after the switch statement in the patch. ---------------------------------------------------------------------- ilja - 15-Sep-04 14:20 CEST ---------------------------------------------------------------------- The auth/authsql.c fix is also in. ---------------------------------------------------------------------- ilja - 15-Sep-04 14:51 CEST ---------------------------------------------------------------------- db.c fixed ---------------------------------------------------------------------- ilja - 15-Sep-04 15:03 CEST ---------------------------------------------------------------------- the fix for mysql/dbmysql.c isn't necessary: mysql_store_result() will return NULL if the query isn't a SELECT query. On top of that mysql_num_fields doesn't work in current versions of MySQL. Bug History Date Modified Username Field Change ====================================================================== 13-Sep-04 23:35ljackson New Bug 13-Sep-04 23:35ljackson File Added: dbmail-2.0rc8.memleakmysql_pop3session.diff 15-Sep-04 12:20ilja Bugnote Added: 0000242 15-Sep-04 12:20ilja Assigned To => ilja 15-Sep-04 12:20ilja Status new => confirmed 15-Sep-04 14:20ilja Bugnote Added: 0000243 15-Sep-04 14:51ilja Bugnote Added: 0000244 15-Sep-04 15:03ilja Bugnote Added: 0000245 ======================================================================
