Memory leak and possible cause of bug #15333
| From: | Joseph Tate | Date: | Mon, 08 Apr 2002 18:42:52 +0000 |
| Subject: | Memory leak and possible cause of bug #15333 | ||
| Groups: | php.dev | ||
| Request: | Send a blank email to php-dev+get-82284@lists.php.net to get a copy of this message | ||
in the copy_zend_constant function it reads:
void copy_zend_constant(zend_constant *c)
{
c->name = zend_strndup(c->name, c->name_len);
if (!(c->flags & CONST_PERSISTENT)) {
zval_copy_ctor(&c->value);
if (c->flags & CONST_EFREE_PERSISTENT) { /* persist_alloc()'d data */
persist_alloc(&c->value);
}
}
}
I draw your attention to the first line in the function:
c->name = zend_strndup(c->name, c->name_len);
First of all, why is this string duplicated only to store it to the same
location? Secondly, is c->name freed somewhere else? Cause I can't see it
being freed. Seems like this line can be removed...
Joseph