Leaking morkCell objects

VERIFIED FIXED in Future

Status

Core Graveyard
Tracking
P3
normal
VERIFIED FIXED
19 years ago
2 years ago

People

(Reporter: Patrick C. Beard, Assigned: Bienvenu)

Tracking

({memory-leak})

Trunk
Future
memory-leak

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

19 years ago
See URL above for allocation point, and attachment for more context. Good luck!
(Reporter)

Comment 1

19 years ago
Created attachment 3314 [details]
annotated stack crawl

Comment 2

19 years ago
Hey Dave, do you have an ETA on these? Thanks,

Warren

Comment 3

19 years ago
There is no ETA.  My current work is on a standalone import utility and I am off
the tree.  Probably hysterisis of a couple days to sync with the tree, partly
because my main machine is 2.5 years old, and partly because by tree is very
stale.  I also have no estimate for time when I will start this bug.
(Assignee)

Updated

19 years ago
Assignee: davidmc → bienvenu
(Assignee)

Comment 4

19 years ago
All this means is the history db wasn't closed, is my guess. I'll look at it.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED

Updated

19 years ago
Target Milestone: M14

Comment 5

19 years ago
M14 ok with you David?
(Assignee)

Comment 6

19 years ago
sure, m14 is fine.
(Assignee)

Comment 7

19 years ago
I'm only seeing these for history, and not for mail - I don't understand why
that would be. I don't see any obvious flaws in the way history is dealing with
rows, but I'll poke around.
(Assignee)

Comment 8

19 years ago
Created attachment 4233 [details] [diff] [review]
proposed patch
(Assignee)

Comment 9

19 years ago
I found two places in history with potential ref-counting problems for cells - I
don't know if the code is executed, but I'll see if the attached patch fixes the
leaks.

Comment 10

19 years ago
looks good. thanks for checking this out. r=waterson

Updated

19 years ago
Keywords: mlk
(Assignee)

Comment 11

19 years ago
moving to m15 - it's a very small leak, and not easily reproducible.
Target Milestone: M14 → M15

Updated

19 years ago
Summary: [MLK] Leaking morkCell objects → Leaking morkCell objects
(Assignee)

Comment 12

18 years ago
moving to m16
Target Milestone: M15 → M16

Comment 13

18 years ago
moving to future.
Target Milestone: M16 → Future
(Assignee)

Comment 14

18 years ago
adding nsbeta3 keyword
Keywords: nsbeta3
(Assignee)

Comment 15

18 years ago
I think this was fixed a long time ago.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
- Per last comments, age of bug, and no reopen - Marking Verified/Fixed.  Please 
reopen if still a problem. 
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.