Re: cvs: php4 /ext/oci8 oci8.c

btw, Derick, I see nothing added into NEWS. Does this work instantly or is the file being built periodically? I remember someone mentioned (Andrei?) that the script was broken. Not sure if it was fixed yet - that's why I was asking. -- Maxim Maletsky maxim@php.net Derick Rethans <derick@php.net> wrote... : > On Thu, 21 Nov 2002, Maxim Maletsky wrote: > > > maxim Thu Nov 21 15:28:11 2002 EDT > > > > Modified files: > > /php4/ext/oci8 oci8.c > > Log: > > Disabled OCIPasswordChange() in safe mode. Possible security risks. > > > > > > + /* Disalllow in Safe Mode */ > > + if (PG(safe_mode)) { > > + RETURN_FALSE; > > + } > > You should add a warning there and not fail without a reason. > > Derick > > -- > > --------------------------------------------------------------------------- > Derick Rethans > http://derickrethans.nl/ > JDI Media Solutions > --------------[ if you hold a unix shell to your ear, do you hear the c? ]- > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php >