[PHP-CVS] cvs: php-src(PHP_5_3) /ext/reflection php_reflection.c

[Antony Dovgal]

tony2001                Thu Sep 27 09:15:10 2007 UTC

  Modified files:              (Branch: PHP_5_3)
    /php-src/ext/reflection     php_reflection.c 
  Log:
  MFH: fix coverity issue #411
  
  
http://cvs.php.net/viewvc.cgi/php-src/ext/reflection/php_reflection.c?r1=1.164.2.33.2.45&r2=1.164.2.33.2.45.2.1&diff_format=u
Index: php-src/ext/reflection/php_reflection.c
diff -u php-src/ext/reflection/php_reflection.c:1.164.2.33.2.45 
php-src/ext/reflection/php_reflection.c:1.164.2.33.2.45.2.1
--- php-src/ext/reflection/php_reflection.c:1.164.2.33.2.45     Mon Aug 20 
17:01:22 2007
+++ php-src/ext/reflection/php_reflection.c     Thu Sep 27 09:15:09 2007
@@ -20,7 +20,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: php_reflection.c,v 1.164.2.33.2.45 2007/08/20 17:01:22 sebastian Exp $ 
*/
+/* $Id: php_reflection.c,v 1.164.2.33.2.45.2.1 2007/09/27 09:15:09 tony2001 
Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -1992,7 +1992,7 @@
 {
        reflection_object *intern;
        parameter_reference *param;
-       zend_class_entry **pce;
+       zend_class_entry **pce, *ce;
 
        METHOD_NOTSTATIC_NUMPARAMS(reflection_parameter_ptr, 0);
        GET_REFLECTION_OBJECT_PTR(param);
@@ -2011,7 +2011,7 @@
                 * lint-mode.
                 */
                if (0 == strncmp(param->arg_info->class_name, "self", 
sizeof("self")- 1)) {
-                       zend_class_entry *ce= param->fptr->common.scope;
+                       ce = param->fptr->common.scope;
                        if (!ce) {
                           zend_throw_exception_ex(reflection_exception_ptr, 0 
TSRMLS_CC, 
                                   "Parameter uses 'self' as type hint but 
function is not a class member!");
@@ -2019,7 +2019,7 @@
                        }
                        pce= &ce;
                } else if (0 == strncmp(param->arg_info->class_name, "parent", 
sizeof("parent")- 1)) {
-                       zend_class_entry *ce= param->fptr->common.scope;
+                       ce = param->fptr->common.scope;
                        if (!ce) {
                           zend_throw_exception_ex(reflection_exception_ptr, 0 
TSRMLS_CC, 
                                   "Parameter uses 'parent' as type hint but 
function is not a class member!");
@@ -4903,7 +4903,7 @@
        php_info_print_table_start();
        php_info_print_table_header(2, "Reflection", "enabled");
 
-       php_info_print_table_row(2, "Version", "$Id: php_reflection.c,v 
1.164.2.33.2.45 2007/08/20 17:01:22 sebastian Exp $");
+       php_info_print_table_row(2, "Version", "$Id: php_reflection.c,v 
1.164.2.33.2.45.2.1 2007/09/27 09:15:09 tony2001 Exp $");
 
        php_info_print_table_end();
 } /* }}} */

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php