Re: [patch] build_id implementation

From: Date: Fri, 16 Jan 2009 17:38:38 +0000
Subject: Re: [patch] build_id implementation
References: 1  Groups: php.internals 
Request: Send a blank email to internals+get-42664@lists.php.net to get a copy of this message
Hi, On Thu, 2009-01-15 at 17:07 -0800, Stanislav Malyshev wrote: > Index: Zend/zend_modules.h [...] > @@ -70,8 +68,7 @@ > struct _zend_module_entry { > unsigned short size; > unsigned int zend_api; > - unsigned char zend_debug; > - unsigned char zts; > + char *build_id; > const struct _zend_ini_entry *ini_entry; > const struct _zend_module_dep *deps; > const char *name; If I got it right it will, when loading a 5.3 extension in 5.2, try to read the extension name from the wrong location and therefor most likely segfault while generating the error message. (it will also try to read zend_debug and zts, of course, but there it should be able to safely read random data) So I'd like to keep the structure in a way that the offset to the name pointer is still he same. johannes

« previous php.internals (#42664) next »