diff options
| author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2019-03-28 14:44:21 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-03-28 14:44:21 (GMT) |
| commit | dffe90ee0eaf77785ad3d4ad7fb3249430ed1cb9 (patch) | |
| tree | 4b71a929afc7dcc6b0419154fb91e00b14dfd4f8 | |
| parent | bb89aa24cf71f9874d1d26f3a2440fefa0b6bbcc (diff) | |
| download | cpython-dffe90ee0eaf77785ad3d4ad7fb3249430ed1cb9.zip cpython-dffe90ee0eaf77785ad3d4ad7fb3249430ed1cb9.tar.gz cpython-dffe90ee0eaf77785ad3d4ad7fb3249430ed1cb9.tar.bz2 | |
bpo-36459: Fix a possible double PyMem_FREE() due to tokenizer.c's tok_nextc() (12601)
Remove the PyMem_FREE() call added in cb90c89. The buffer will be
freed when PyTokenizer_Free() is called on the tokenizer state.
(cherry picked from commit cda139d1ded6708665b53e4ed32ccc1d2627e1da)
Co-authored-by: Zackery Spytz <zspytz@gmail.com>
| -rw-r--r-- | Misc/NEWS.d/next/Core and Builtins/2019-03-27-22-35-16.bpo-36459.UAvkKp.rst | 1 | ||||
| -rw-r--r-- | Parser/tokenizer.c | 1 |
2 files changed, 1 insertions, 1 deletions
diff --git a/Misc/NEWS.d/next/Core and Builtins/2019-03-27-22-35-16.bpo-36459.UAvkKp.rst b/Misc/NEWS.d/next/Core and Builtins/2019-03-27-22-35-16.bpo-36459.UAvkKp.rst new file mode 100644 index 0000000..6c234a6 --- /dev/null +++ b/Misc/NEWS.d/next/Core and Builtins/2019-03-27-22-35-16.bpo-36459.UAvkKp.rst @@ -0,0 +1 @@ +Fix a possible double ``PyMem_FREE()`` due to tokenizer.c's ``tok_nextc()``. diff --git a/Parser/tokenizer.c b/Parser/tokenizer.c index 6d7869c..8966661 100644 --- a/Parser/tokenizer.c +++ b/Parser/tokenizer.c @@ -979,7 +979,6 @@ tok_nextc(register struct tok_state *tok) newbuf = (char *)PyMem_REALLOC(newbuf, newsize); if (newbuf == NULL) { - PyMem_FREE(tok->buf); tok->done = E_NOMEM; tok->cur = tok->inp; return EOF; |