http://bugs.winehq.org/show_bug.cgi?id=20334
--- Comment #10 from Nikolay Sivov bunglehead@gmail.com 2009-10-13 04:25:01 --- (In reply to comment #9)
(In reply to comment #6)
A small comment here. Could you reduce the buffer size? Or allocating 8K really means something?
It doen't means something for wine, but under XP with 95, 98, Me compatability mode, new allocated string is outside of previous if CHUNK_SIZE is less than 16384. Range test fails, so I cannot assert is null terminator always present. But, with 16394, range test pass, new string allocated on the old string with trash, and new string is null terminated.
Didn't know that, if it will fail on native you should use required 8K size of course.
(In reply to comment #7)
One further small comment: any tests that currently fail on wine need to be preceded with todo_wine.
I provide bug fix and test. Test with bug fix doesn't fail. Should I add todo_wine?
Of course not, patch shouldn't bring new failures including success inside todo_wine.
Please review another version of patch.
Send it to wine-patches for review.