#39260 [NEW]: A misleading bit in the Zend API
| From: | sfox@php.net | Date: | Thu, 26 Oct 2006 01:57:41 +0000 |
| Subject: | #39260 [NEW]: A misleading bit in the Zend API | ||
| Groups: | php.bugs | ||
| Request: | Send a blank email to php-bugs+get-103992@lists.php.net to get a copy of this message | ||
From: sfox@php.net
Operating system: irrelevant
PHP version: 5.2.0RC5
PHP Bug Type: Scripting Engine problem
Bug description: A misleading bit in the Zend API
Description:
------------
Not the biggest bug in the world, more a mosquito...
In zend_compile.h:
ZEND_API int zend_unmangle_property_name(char *mangled_property, int
mangled_property_len, char **prop_name, char **class_name);
In zend_compile.c:
ZEND_API int zend_unmangle_property_name(char *mangled_property, int len,
char **class_name, char **prop_name)
Someone's obviously noticed this a long time ago - on SUCCESS we have:
*class_name = mangled_property+1;
*prop_name = (*class_name)+class_name_len;
and nobody writing for PECL seems to have made the obvious mistake (yet).
But somebody, somewhere, will.
The declaration in the header file obviously needs changing to match the
source file, and as for the implementation - well, it might just be nice
to make it read more sanely :) If I trusted myself to do that without
somehow messing it up along the way and breaking practically every
extension out there, there'd be a patch with this report. I'm afraid I
don't, sorry.
--
Edit bug report at http://bugs.php.net/?id=39260&edit=1
--
Try a CVS snapshot (PHP 4.4): http://bugs.php.net/fix.php?id=39260&r=trysnapshot44
Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=39260&r=trysnapshot52
Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=39260&r=trysnapshot60
Fixed in CVS: http://bugs.php.net/fix.php?id=39260&r=fixedcvs
Fixed in release: http://bugs.php.net/fix.php?id=39260&r=alreadyfixed
Need backtrace: http://bugs.php.net/fix.php?id=39260&r=needtrace
Need Reproduce Script: http://bugs.php.net/fix.php?id=39260&r=needscript
Try newer version: http://bugs.php.net/fix.php?id=39260&r=oldversion
Not developer issue: http://bugs.php.net/fix.php?id=39260&r=support
Expected behavior: http://bugs.php.net/fix.php?id=39260&r=notwrong
Not enough info: http://bugs.php.net/fix.php?id=39260&r=notenoughinfo
Submitted twice: http://bugs.php.net/fix.php?id=39260&r=submittedtwice
register_globals: http://bugs.php.net/fix.php?id=39260&r=globals
PHP 3 support discontinued: http://bugs.php.net/fix.php?id=39260&r=php3
Daylight Savings: http://bugs.php.net/fix.php?id=39260&r=dst
IIS Stability: http://bugs.php.net/fix.php?id=39260&r=isapi
Install GNU Sed: http://bugs.php.net/fix.php?id=39260&r=gnused
Floating point limitations: http://bugs.php.net/fix.php?id=39260&r=float
No Zend Extensions: http://bugs.php.net/fix.php?id=39260&r=nozend
MySQL Configuration Error: http://bugs.php.net/fix.php?id=39260&r=mysqlcfg