Last Comment Bug 673052 - Memory leak in the shell's new option parsing code
: Memory leak in the shell's new option parsing code
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla8
Assigned To: Chris Leary [:cdleary] (not checking bugmail)
:
:
Mentors:
Depends on:
Blocks: 668095
  Show dependency treegraph
 
Reported: 2011-07-20 23:47 PDT by Nicholas Nethercote [:njn]
Modified: 2011-07-22 14:07 PDT (History)
4 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Free arguments. (929 bytes, patch)
2011-07-21 12:14 PDT, Chris Leary [:cdleary] (not checking bugmail)
dvander: review+
Details | Diff | Splinter Review

Description Nicholas Nethercote [:njn] 2011-07-20 23:47:24 PDT
==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==
Comment 1 Chris Leary [:cdleary] (not checking bugmail) 2011-07-21 12:14:53 PDT
Created attachment 547464 [details] [diff] [review]
Free arguments.

Note You need to log in before you can comment on or make changes to this bug.