Skip to content

Commit

Permalink
bpo-35720: Fixing a memory leak in pymain_parse_cmdline_impl() (GH-11528
Browse files Browse the repository at this point in the history
)

When the loop in the pymain_read_conf function in this same file
calls pymain_init_cmdline_argv() a 2nd time, the pymain->command
buffer of wchar_t is overriden and the previously allocated memory
is never freed.
  • Loading branch information
Lucas-C authored and vstinner committed Jan 22, 2019
1 parent 9932a22 commit 35ca182
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fixed a minor memory leak in pymain_parse_cmdline_impl function in Modules/main.c
1 change: 1 addition & 0 deletions Modules/main.c
Original file line number Diff line number Diff line change
Expand Up @@ -1376,6 +1376,7 @@ pymain_read_conf(_PyMain *pymain, _PyCoreConfig *config,
goto done;
}
pymain_clear_cmdline(pymain, cmdline);
pymain_clear_pymain(pymain);
memset(cmdline, 0, sizeof(*cmdline));
config->utf8_mode = new_utf8_mode;
config->coerce_c_locale = new_coerce_c_locale;
Expand Down

0 comments on commit 35ca182

Please sign in to comment.