Closed Bug 456894 Opened 14 years ago Closed 14 years ago can't handle negative leaks


(Testing :: Mochitest, defect)

Not set


(Not tracked)



(Reporter: Dolske, Assigned: Dolske)




(1 file, 1 obsolete file)

While investigating some failures, I found some cases where complains "missing output line for total leaks!", when it's clearly in the output.

Example from

Scroll up and you'll find:

   0 TOTAL                                          -4    63920 -1005666652 ....

Looks like the problem is the regex uses to find the "TOTAL" line. It's not expecting negative numbers. dbaron says this can happen when bogus addref/release patterns confuse tracerefcount.

The script should catch these, and handle as a "omgwtf something is wrong" case.
Attached patch Patch v.1 (obsolete) — Splinter Review
How convenient, dbaron is a Test Harness peer. :-)

[I should note I haven't run this yet, but it seems like an obvious enough change.]
Assignee: nobody → dolske
Attachment #340420 - Flags: review?(dbaron)
Attached patch Patch v.2Splinter Review
Added another check for one of the columns that might be negative, and tested that it works with and without negative values in the output.
Attachment #340420 - Attachment is obsolete: true
Attachment #340472 - Flags: review?(dbaron)
Attachment #340420 - Flags: review?(dbaron)
Comment on attachment 340472 [details] [diff] [review]
Patch v.2

r=dbaron, although you might want to run this by whoever owns
Attachment #340472 - Flags: review?(dbaron) → review+
Attachment #340472 - Flags: review?(ted.mielczarek)
Attachment #340472 - Flags: review?(ted.mielczarek) → review+
Pushed changeset 5fabcd7363bf.
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b2
Component: Testing → Mochitest
Product: Core → Testing
QA Contact: testing → mochitest
Target Milestone: mozilla1.9.1b2 → ---
Version: Trunk → unspecified
Depends on: 482236
You need to log in before you can comment on or make changes to this bug.