Closed Bug 319710 Opened 19 years ago Closed 19 years ago

mozMySpell::Suggest leaks in several places

Categories

(Core :: Spelling checker, defect)

Product:
Core
Component:
Spelling checker
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: mscott)

References

Details

(Keywords: memory-leak)

1) We never free |wlst|, which is allocated by suggest(). So we always leak it. 2) If allocating |tmpPtr| fails, we also leak all suggestions in |wlst|. 3) If allocating |dest| fails, we leak (in addition to |wlst|) all remaining suggestions in |wlst|, leak |tmpPtr|, and leak all the suggestions we've placed in |tmpPtr| so far. 4) If Convert() fails (and this CAN happen easily for invalid input!) we leak the same things as in #3 plus the |dest| we just allocated.
Flags: blocking1.9a1?
Flags: blocking1.8.1?
Flags: blocking1.8.0.1?
this is being tracked in another bug which is already nominated. Clearing the flags.
Flags: blocking1.9a1?
Flags: blocking1.8.1?
Flags: blocking1.8.0.1?
Which bug? Add a dependency?
Depends on: 307052
fixed by 307052
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.