Bug 11286 - Allocator mismatch in CookieJarWin
Summary: Allocator mismatch in CookieJarWin
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 420+
Hardware: PC Windows XP
: P2 Trivial
Assignee: Darin Fisher (:fishd, Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-13 18:29 PDT by Darin Fisher (:fishd, Google)
Modified: 2006-10-31 10:52 PST (History)
0 users

See Also:

Attachments
patch (1.58 KB, patch)
2006-10-13 18:31 PDT, Darin Fisher (:fishd, Google) 		mjs: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Fisher (:fishd, Google) 2006-10-13 18:29:28 PDT 
Allocator mismatch in CookieJarWin:

  UChar* buffer = new UChar[count];
  ...
  delete buffer;

That should be "delete[] buffer" though this surely doesn't cause any real problem since UChar doesn't have a destructor.  Anyways, patch coming up...
Comment 1 Darin Fisher (:fishd, Google) 2006-10-13 18:31:00 PDT 
Created attachment 11084 [details]
patch
Comment 2 Maciej Stachowiak 2006-10-31 04:42:13 PST 
Comment on attachment 11084 [details]
patch

r=me

Thanks also for cleaning up the style issues. This should be clear to land.
Comment 3 Alexey Proskuryakov 2006-10-31 10:52:32 PST 
Committed revision 17485.