Re: gawk: free_wstr() invalid write

[Andrew J. Schorr]

On Mon, Dec 18, 2006 at 06:27:52PM +0300, Dmitry V. Levin wrote:
> free_wstr() zeroes wstptr and wstlen pointers even if WSTRCUR flag is not
> set, which is wrong and may lead to invalid write, similar to invalid free
> in str2wstr().
> 
> Proposed fix is attached.

FYI, your str2str and free_wstr patches seem correct to me,
and they resemble the code in the xgawk source tree:

   http://sourceforge.net/projects/xmlgawk/

In particular:

   http://xmlgawk.cvs.sourceforge.net/xmlgawk/xmlgawk/node.c?revision=1.5

(except the 'free_wstr' function is named 'release_wstr' in xgawk
at the moment).

Regards,
Andy