Closed Bug 287383 Opened 20 years ago Closed 20 years ago

heap corruption from ReplaceChar/StripChars

Categories

(Core Graveyard :: Embedding: MFC Embed, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 obsolete file)

steps:
1. run mfcembed (debug build)
2. right click a link and select save as

expected results: no asserts from msvcrt(d)

actual results:
 _CrtIsValidHeapPointer(pUserData)

outputdebugstring tends to list something like:
HEAP[mfcembed.exe]: Invalid Address specified to RtlValidateHeap( 003F0000,
019E451B )
Attachment #178362 - Flags: superreview?(dbaron)
Attachment #178362 - Flags: review?(dbaron)
Comment on attachment 178362 [details] [diff] [review]
keep track of the allocated pointer

darin wrote this, so you should try requesting review from him instead
Attachment #178362 - Flags: superreview?(dbaron)
Attachment #178362 - Flags: superreview?(darin)
Attachment #178362 - Flags: review?(dbaron)
Attachment #178362 - Flags: review?(darin)
Comment on attachment 178362 [details] [diff] [review]
keep track of the allocated pointer

oops
Attachment #178362 - Flags: superreview?(darin)
Attachment #178362 - Flags: superreview+
Attachment #178362 - Flags: review?(darin)
Attachment #178362 - Flags: review+
Comment on attachment 178362 [details] [diff] [review]
keep track of the allocated pointer

mozilla/embedding/tests/mfcembed/BrowserView.cpp	1.51
Attachment #178362 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: