# [PATCH]: Fix for endless loop in PDOStatement::debugDumpParams()

- Thread: https://externals.io/message/41626
- Original archive: https://news-web.php.net/php.internals/41626
- Started by Jonah H. Harris on 2008-11-03T18:52:21+00:00
- Messages: 5
- 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

---

## #41626 — Jonah H. Harris — 2008-11-03T18:52:21+00:00

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

While using PDOStatement::debugDumpParams, I noticed that it results
in an endless loop because the hash table is not being traversed.  As
such, attached is a patch against php5 HEAD which adds
zend_hash_move_forward_ex accordingly.

-- 
Jonah H. Harris, Senior DBA
myYearbook.com

---

## #41627 — Scott MacVicar — 2008-11-03T18:54:45+00:00

URL: https://externals.io/message/41626#41627 · In reply to: #41626

Jonah H. Harris wrote:
> While using PDOStatement::debugDumpParams, I noticed that it results
> in an endless loop because the hash table is not being traversed.  As
> such, attached is a patch against php5 HEAD which adds
> zend_hash_move_forward_ex accordingly.
> 
> 

Your patch was stripped, can you attach against with a .txt extension.

Scott

---

## #41628 — Lukas Kahwe Smith — 2008-11-03T19:15:14+00:00

URL: https://externals.io/message/41626#41628 · In reply to: #41627

> Jonah H. Harris wrote:
> > […]
>
> Your patch was stripped, can you attach against with a .txt extension.


interesting .. i wasnt aware of this method. there is a doc bug on  
this on pecl.php.net:
http://pecl.php.net/bugs/bug.php?id=13035

once again a reminder that we really do not have a good setup yet to  
manage the movement of extensions between pecl and php-src. also a  
pitty that the GSoC for a unified bug tracker apparently did not get  
finished (?)

regards,
Lukas Kahwe Smith
mls@pooteeweet.org

---

## #41630 — Jonah H. Harris — 2008-11-03T19:44:13+00:00

URL: https://externals.io/message/41626#41630 · In reply to: #41627

> Your patch was stripped, can you attach against with a .txt extension.

Attached.

-- 
Jonah H. Harris, Senior DBA
myYearbook.com

---

## #41632 — Felipe Pena — 2008-11-03T20:57:09+00:00

URL: https://externals.io/message/41626#41632 · In reply to: #41630

Hi Jonah,

Em Seg, 2008-11-03 às 14:44 -0500, Jonah H. Harris escreveu:
> 
> > […]
> 
> Attached.
> 

Thanks for the patch!

Fixed in CVS. (5.2.7+)

-- 
Regards,
Felipe Pena