[webkit-reviews] review denied: [Bug 11414] [s60]China Variant Crashes : [Attachment 11438] modified as suggested

[bugzilla-request-daemon at macosforge.org]

Zalan Bujtas <zbujtas at gmail.com> has denied Zalan Bujtas <zbujtas at gmail.com>'s
request for review:
Bug 11414: [s60]China Variant Crashes
http://bugs.webkit.org/show_bug.cgi?id=11414

Attachment 11438: modified as suggested
http://bugs.webkit.org/attachment.cgi?id=11438&action=edit

------- Additional Comments from Zalan Bujtas <zbujtas at gmail.com>
1, ptr.Length() + tmpBuf.Length() should be passed to checkRealloc instead of
tmpBuf.Length()
2, outBuf->Des().Copy( aPtr ) should be used instead of
outBuf->operator=(aPtr);
as the buffer that aPtr points to is destroyed right after checkRealloc returns
(delete outBuf;)
3, checkRealloc should not leave, it should return null in case of oom.