# [PATCH] fixes #26194

- Thread: https://externals.io/message/5659
- Original archive: https://news-web.php.net/php.internals/5659
- Started by Frank M. Kromann on 2003-11-22T05:08:51+00:00
- Messages: 3
- 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

---

## #5659 — Frank M. Kromann — 2003-11-22T05:08:51+00:00

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

Take a look at iconv.h line 80

#define iconv libiconv

This line is the problem I'll change the php_iconv.h file to include.

- Frank

That will fix it
> 
> 
> 
> > […]
too
> > […]
> 
> I don't think this is Windows specific issue. From my experience, this
> might be caused by random heap corruption. (most likely spotted in
> another extension) And there's never been a macro like "iconv" or 
> whatever
> in the standard MS C library.
> 
> Moriyoshi

---

## #5661 — Moriyoshi Koizumi — 2003-11-22T08:10:38+00:00

URL: https://externals.io/message/5659#5661 · In reply to: #5659

> Take a look at iconv.h line 80
>
> #define iconv libiconv
>
> This line is the problem I'll change the php_iconv.h file to include.

You must be right. Seems I overlooked that stuff totally though I
recognised the evil #define is there. I'll commit a patch for this
shortly if you don't mind. Thanks.

Moriyoshi

---

## #5663 — Steph — 2003-11-22T13:33:15+00:00

URL: https://externals.io/message/5659#5663 · In reply to: #5659

erm, I did ... but that line wasn't the thing that broke this.  

(ZEND_FENTRY was.)

> -----Original Message-----
> From: Frank M. Kromann [mailto:frank@kromann.info]
> Sent: 22 November 2003 05:09
> To: Moriyoshi Koizumi
> Cc: Steph; internals; Derick Rethans
> Subject: Re: [PHP-DEV] [PATCH] fixes #26194
> 
> 
> Take a look at iconv.h line 80
> 
> #define iconv libiconv
> 
> This line is the problem I'll change the php_iconv.h file to include.
> 
> - Frank
> 
> That will fix it
> > […]
> too
> > […]