The default bug view has changed. See this FAQ.

Memory leak in the shell's new option parsing code

RESOLVED FIXED in mozilla8

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: njn, Assigned: cdleary)

Tracking

unspecified
mozilla8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
==4957== 28 bytes in 1 blocks are definitely lost in loss record 1 of 2
==4957==    at 0x48D9876: malloc (vg_replace_malloc.c:236)
==4957==    by 0x805BA59: js_malloc (jsutil.h:237)
==4957==    by 0x805E411: js::cli::detail::StringOption* js::OffTheBooks::new_<js::cli::detail::StringOption, int, char const*, char const*, char const*>(int, char const*, char const*, char const*) (in /home/njn/moz/mi3/js/src/d32/shell/js)
==4957==    by 0x805D378: js::cli::OptionParser::addOptionalStringArg(char const*, char const*) (jsoptparse.cpp:609)
==4957==    by 0x8057CF1: main (js.cpp:5957)
==4957== 
==4957== 52 bytes in 1 blocks are definitely lost in loss record 2 of 2
==4957==    at 0x48D9876: malloc (vg_replace_malloc.c:236)
==4957==    by 0x805BA59: js_malloc (jsutil.h:237)
==4957==    by 0x805E4F2: js::cli::detail::MultiStringOption* js::OffTheBooks::new_<js::cli::detail::MultiStringOption, int, char const*, char const*, char const*>(int, char const*, char const*, char const*) (in /home/njn/moz/mi3/js/src/d32/shell/js)
==4957==    by 0x805D465: js::cli::OptionParser::addOptionalMultiStringArg(char const*, char const*) (jsoptparse.cpp:623)
==4957==    by 0x8057D18: main (js.cpp:5960)
==4957==
Blocks: 668095
Created attachment 547464 [details] [diff] [review]
Free arguments.
Assignee: general → cdleary
Status: NEW → ASSIGNED
Attachment #547464 - Flags: review?(dvander)
Attachment #547464 - Flags: review?(dvander) → review+
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/cb9b241dc50e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.