Closed Bug 823782 Opened 7 years ago Closed 6 years ago

Valgrind on tbpl detects leak - 120 bytes are definitely lost (direct) with js::ctypes::Library::Declare or js::ctypes::StructType::Create or js::ctypes::Int64Base::Construct on the stack

Categories

(Core :: JavaScript Engine, defect, major)

All
Linux
defect
Not set
major

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: gkw, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: memory-leak, regression, valgrind)

Attachments

(3 files)

Attached file Valgrind stack
Valgrind detects a leak of 120 bytes (direct) with js::ctypes::Library::Declare on the stack, see attached snippet which comes from:

https://tbpl.mozilla.org/php/getParsedLog.php?id=18125018&tree=Firefox&full=1

Regression window:

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=553a3bcf1fe7&tochange=edd45de440ba
Suppressed in Valgrind tbpl runs in https://hg.mozilla.org/mozilla-central/rev/beec8736b9ec
Also, this seems 64-bit only, 32-bit Linux Valgrind tbpl runs do not seem to see this issue.
(In reply to Gary Kwong [:gkw] from comment #2)
> Also, this seems 64-bit only, 32-bit Linux Valgrind tbpl runs do not seem to
> see this issue.

32-bit runs have the same issue, and variants also show js::ctypes::StructType::Create on the stack.
Hardware: x86_64 → All
Summary: Valgrind on tbpl detects leak - 120 bytes are definitely lost (direct) with js::ctypes::Library::Declare on the stack → Valgrind on tbpl detects leak - 120 bytes are definitely lost (direct) with js::ctypes::Library::Declare or js::ctypes::StructType::Create on the stack
Here's another variant, this time involving js::ctypes::Int64Base::Construct.
Summary: Valgrind on tbpl detects leak - 120 bytes are definitely lost (direct) with js::ctypes::Library::Declare or js::ctypes::StructType::Create on the stack → Valgrind on tbpl detects leak - 120 bytes are definitely lost (direct) with js::ctypes::Library::Declare or js::ctypes::StructType::Create or js::ctypes::Int64Base::Construct on the stack
This is no longer occurring in Valgrind-on-TBPL runs.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.