cvs: php-src / NEWS /ext/com_dotnet com_extension.c com_wrapper.c php_com_dotnet.h

wharmby Mon Mar 5 12:56:26 2007 UTC Modified files: /php-src/ext/com_dotnet com_wrapper.c com_extension.c php_com_dotnet.h /php-src NEWS Log: Fixed bug #35872 (Prevent object store references during RSHUTDOWN) http://cvs.php.net/viewvc.cgi/php-src/ext/com_dotnet/com_wrapper.c?r1=1.14&r2=1.15&diff_format=u Index: php-src/ext/com_dotnet/com_wrapper.c diff -u php-src/ext/com_dotnet/com_wrapper.c:1.14 php-src/ext/com_dotnet/com_wrapper.c:1.15 --- php-src/ext/com_dotnet/com_wrapper.c:1.14 Sat Feb 24 16:25:53 2007 +++ php-src/ext/com_dotnet/com_wrapper.c Mon Mar 5 12:56:26 2007 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: com_wrapper.c,v 1.14 2007/02/24 16:25:53 helly Exp $ */ +/* $Id: com_wrapper.c,v 1.15 2007/03/05 12:56:26 wharmby Exp $ */ /* This module exports a PHP object as a COM object by wrapping it * using IDispatchEx */ @@ -92,13 +92,17 @@ # define TSRMLS_FIXED() #endif -#define FETCH_DISP(methname) \ - TSRMLS_FIXED() \ - php_dispatchex *disp = (php_dispatchex*)This; \ - trace(" PHP:%s %s\n", Z_OBJCE_P(disp->object)->name, methname); \ - if (GetCurrentThreadId() != disp->engine_thread) \ - return RPC_E_WRONG_THREAD; - +#define FETCH_DISP(methname) \ + TSRMLS_FIXED() \ + php_dispatchex *disp = (php_dispatchex*)This; \ + if (COMG(rshutdown_started)) { \ + trace(" PHP Object:%p (name:unknown) %s\n", disp->object, methname); \ + } else { \ + trace(" PHP Object:%p (name:%s) %s\n", disp->object, Z_OBJCE_P(disp->object)->name, methname); \ + } \ + if (GetCurrentThreadId() != disp->engine_thread) { \ + return RPC_E_WRONG_THREAD; \ + } static HRESULT STDMETHODCALLTYPE disp_queryinterface( IDispatchEx *This, @@ -534,7 +538,7 @@ { php_dispatchex *disp = (php_dispatchex*)CoTaskMemAlloc(sizeof(php_dispatchex)); - trace("constructing a COM proxy\n"); + trace("constructing a COM wrapper for PHP object %p (%s)\n", object, Z_OBJCE_P(object)->name); if (disp == NULL) return NULL; @@ -559,8 +563,13 @@ { TSRMLS_FETCH(); - trace("destroying COM wrapper for PHP object %s\n", Z_OBJCE_P(disp->object)->name); - + /* Object store not available during request shutdown */ + if (COMG(rshutdown_started)) { + trace("destroying COM wrapper for PHP object %p (name:unknown)\n", disp->object); + } else { + trace("destroying COM wrapper for PHP object %p (name:%s)\n", disp->object, Z_OBJCE_P(disp->object)->name); + } + disp->id = 0; if (disp->refcount > 0) http://cvs.php.net/viewvc.cgi/php-src/ext/com_dotnet/com_extension.c?r1=1.24&r2=1.25&diff_format=u Index: php-src/ext/com_dotnet/com_extension.c diff -u php-src/ext/com_dotnet/com_extension.c:1.24 php-src/ext/com_dotnet/com_extension.c:1.25 --- php-src/ext/com_dotnet/com_extension.c:1.24 Mon Jan 1 09:29:21 2007 +++ php-src/ext/com_dotnet/com_extension.c Mon Mar 5 12:56:26 2007 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: com_extension.c,v 1.24 2007/01/01 09:29:21 sebastian Exp $ */ +/* $Id: com_extension.c,v 1.25 2007/03/05 12:56:26 wharmby Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -315,6 +315,7 @@ */ PHP_RINIT_FUNCTION(com_dotnet) { + COMG(rshutdown_started) = 0; return SUCCESS; } /* }}} */ @@ -328,6 +329,7 @@ php_com_dotnet_rshutdown(TSRMLS_C); } #endif + COMG(rshutdown_started) = 1; return SUCCESS; } /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/ext/com_dotnet/php_com_dotnet.h?r1=1.7&r2=1.8&diff_format=u Index: php-src/ext/com_dotnet/php_com_dotnet.h diff -u php-src/ext/com_dotnet/php_com_dotnet.h:1.7 php-src/ext/com_dotnet/php_com_dotnet.h:1.8 --- php-src/ext/com_dotnet/php_com_dotnet.h:1.7 Mon Jan 1 09:29:22 2007 +++ php-src/ext/com_dotnet/php_com_dotnet.h Mon Mar 5 12:56:26 2007 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: php_com_dotnet.h,v 1.7 2007/01/01 09:29:22 sebastian Exp $ */ +/* $Id: php_com_dotnet.h,v 1.8 2007/03/05 12:56:26 wharmby Exp $ */ #ifndef PHP_COM_DOTNET_H #define PHP_COM_DOTNET_H @@ -47,6 +47,7 @@ zend_bool autoreg_case_sensitive; void *dotnet_runtime_stuff; /* opaque to avoid cluttering up other modules */ int code_page; /* default code_page if left unspecified */ + zend_bool rshutdown_started; ZEND_END_MODULE_GLOBALS(com_dotnet) #ifdef ZTS http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2140&r2=1.2141&diff_format=u Index: php-src/NEWS diff -u php-src/NEWS:1.2140 php-src/NEWS:1.2141 --- php-src/NEWS:1.2140 Fri Feb 16 19:36:45 2007 +++ php-src/NEWS Mon Mar 5 12:56:26 2007 @@ -58,5 +58,6 @@ - Added shm_has_var() function. (Mike) - Added str_getcsv() function. (Sara) - Added ext/hash support to ext/session's ID generator. (Sara) +- Fixed bug #35872 (Prevent object store references during RSHUTDOWN) (Andy)