# tests/lang/bug24773.phpt, segfault with 4.3.5RC2

- Thread: https://externals.io/message/7617
- Original archive: https://news-web.php.net/php.internals/7617
- Started by Pierre-Alain Joye on 2004-02-05T15:24:56+00:00
- Messages: 4
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #7617 — Pierre-Alain Joye — 2004-02-05T15:24:56+00:00

URL: https://externals.io/message/7617#7617

Hello,

I got segfaults with the test tests/lang/bug24773.phpt. It seems the
test added in HEAD has been either lost or never applied.

Please find an attempt to fix as attachment, feel free to drop it and
apply your own and cleaner one ;-)

hth

pierre

---

## #7619 — Pierre-Alain Joye — 2004-02-05T16:22:49+00:00

URL: https://externals.io/message/7617#7619 · In reply to: #7617

About my previous post. This commit has been lost.
And the test:
 (EX(Ts)[EX(opline)->result.u.var].EA.type ==IS_STRING_OFFSET)

is a bit cleaner than mine.

pierre

On Wed, 03 Dec 2003 19:33:17 -0000
moriyoshi@php.net (Moriyoshi Koizumi) wrote:

> moriyoshi		Wed Dec  3 14:33:17 2003 EDT
> 
>   Modified files:              (Branch: PHP_4_3)
>     /Zend	zend_execute.c 
>   Log:
>   MFZE2(r-1.557): Fix bug #24773 (unset()ing string offsets crashes
>   PHP)

---

## #7625 — Moriyoshi Koizumi — 2004-02-06T12:44:48+00:00

URL: https://externals.io/message/7617#7625 · In reply to: #7619

> About my previous post. This commit has been lost.
> And the test:
>  (EX(Ts)[EX(opline)->result.u.var].EA.type ==IS_STRING_OFFSET)
>
> is a bit cleaner than mine.

My patch was reverted because it had a flaw and didn't address the whole
issue. DmItry already resolved it in ZE2 with the following patch.

I think this should be backported partly before 4.3.5 gets out of the
door.

http://news.php.net/article.php?group=php.zend-engine.cvs&article=2118

> pierre

Moriyoshi

---

## #7626 — Pierre-Alain Joye — 2004-02-06T12:47:53+00:00

URL: https://externals.io/message/7617#7626 · In reply to: #7625

On Fri, 6 Feb 2004 21:44:48 +0900
Moriyoshi Koizumi <moriyoshi@at.wakwak.com> wrote:

> I think this should be backported partly before 4.3.5 gets out of the
> door.

Ilia MFH'd it. Looks like my patch was good, but in the
future I will rather check the cvs logs before trying anything ;)

pierre