Bug #10252 Updated: strcat() used on uninitialized string

ID: 10252 Updated by: joey Reported By: dzoll@nyscul.org Old-Status: Analyzed Status: Closed Bug Type: ODBC related PHP Version: 4.0.4pl1 Assigned To: Comments: Closed in CVS. I actually used strlcpy() instead, but same general concept. Previous Comments: --------------------------------------------------------------------------- [2001-04-16 23:10:35] kalowsky@php.net Joey Smith has(had?) a patch he was considering applying that should fix this problem as well... waiting to hear back from him currently on it's status... --------------------------------------------------------------------------- [2001-04-09 15:30:56] dzoll@nyscul.org I was getting intermittent errors connecting to databases. Sometimes it would work fine, sometimes my connect string would turn to gibberish and sometimes the process would segfault. On further examination, in ext/odbc/php_odbc.c:odbc_sqlconnect(), within the if(strstr(char *)db,";") block, there is a line "strcat(ldb,db);". At this point in the code, ldb was just emalloced, and had uninitialized contents. I think strcpy() is more appopriate here, and changing the code to this seems to have cleared up my errors. Thank you very much. --------------------------------------------------------------------------- ATTENTION! Do NOT reply to this email! To reply, use the web interface found at http://bugs.php.net/?id=10252&edit=2