#22437 [Opn->Csd]: Segmentation violation

ID: 22437 Updated by: sniper@php.net Reported By: olle at compound dot se -Status: Open +Status: Closed Bug Type: ODBC related Operating System: Redhat7.2/kernel 2.4.9 PHP Version: 4.3.1 Assigned To: kalowsky New Comment: But this odbc related bug is now fixed -> closed. Previous Comments: ------------------------------------------------------------------------ [2003-03-10 01:54:12] olle at compound dot se Thank you for your prompt answers. I have patched the original 4.3.1 php_odbc.c as suggested in the "[26 Feb 9:59am CST] mboeren@php.net"-note, and the Valgrind "invalid write"-messages did go away. We also found memory misuse in the UnixODBC code, which we have corrected. Things are running better, but we are still experiencing segmentation violations which we think originate from the Sablot XML-package. ------------------------------------------------------------------------ [2003-03-09 19:14:57] sniper@php.net No feedback was provided. The bug is being suspended because we assume that you are no longer experiencing the problem. If this is not the case and you are able to provide the information that was requested earlier, please do so and change the status of the bug back to "Open". Thank you. ------------------------------------------------------------------------ [2003-03-04 10:35:24] iliaa@php.net Please try using this CVS snapshot: http://snaps.php.net/php4-STABLE-latest.tar.gz For Windows: http://snaps.php.net/win32/php4-win32-STABLE-latest.zip ------------------------------------------------------------------------ [2003-02-26 15:32:16] iliaa@php.net Try applying the patch at (againt latest stable CVS): http://bb.prohost.org/odbc43.txt If the problem is the result of sprintf() overflow this should correct it. ------------------------------------------------------------------------ [2003-02-26 09:59:02] mboeren@php.net From php_odbc.c: void odbc_do_connect(INTERNAL_FUNCTION_PARAMETERS, int persistent) ... len = strlen(db) + strlen(uid) + strlen(pwd) + sizeof(ODBC_TYPE) + 5; hashed_details = emalloc(len); hashed_len = sprintf(hashed_details, "%s_%s_%s_%s_%d", ODBC_TYPE, db, uid, pwd, cur_opt); ==> shouldn't len include the length of the string representation of cur_opt? Hope this helps, Marc. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/22437 -- Edit this bug report at http://bugs.php.net/?id=22437&edit=1