# Suggested fix for bug #22592

- Thread: https://externals.io/message/62
- Original archive: https://news-web.php.net/php.internals/62
- Started by Moriyoshi Koizumi on 2003-03-16T00:07:49+00:00
- Messages: 2
- 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

---

## #62 — Moriyoshi Koizumi — 2003-03-16T00:07:49+00:00

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

Attached is a suggested fix for bug #22592, which has finally come out of 
a successful long-hour-introspection with zend_execute.c.

Although the patch actually does somewhat hackish things and should 
appear to be invalid, I believe it's the right one.

Moriyoshi

---

## #63 — Andi Gutmans — 2003-03-16T05:20:29+00:00

URL: https://externals.io/message/62#63 · In reply to: #62

It indeed does look very hackish. Is this bug the same bug as the last 
patch of yours addressed?
I'll try and take a look at the bug report this week. I'm not quite sure 
this can/should be fixed. AFAIK the return value of $str[1] = '*' is the 
updated string itself and not the assigned '*'.

Andi

At 09:07 AM 3/16/2003 +0900, Moriyoshi Koizumi wrote:
>Attached is a suggested fix for bug #22592, which has finally come out of
>a successful long-hour-introspection with zend_execute.c.
>
>Although the patch actually does somewhat hackish things and should
>appear to be invalid, I believe it's the right one.
>
>Moriyoshi