You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
parsetok(): Assertion `(intptr_t)(int)(a - line_start) == (a - line_start)' failed, when running get-pip.py #81784
Reproducing depends if REALLOC() returns a new pointer, see below.
I attached get-pip2.py: simplified version which crash when using "./python get-pip2.py --help". If it doesn't crash, duplicate lines to make the file larger.
--
The root issue seems to be that tok->multi_line_start isn't updated on REALLOC() in tok_nextc():
This is marked as release blocker but since BPO-37433 is still in review, I'm releasing 3.0b3 as is. Please resolve this by b4, I will block the last beta on this issue.
This is marked as release blocker but since BPO-37433 is still in review, I'm releasing 3.0b3 as is. Please resolve this by b4, I will block the last beta on this issue.
Activity
vstinner commentedon Jul 16, 2019
bpo-16806 introduced a regression with the following change:
commit 995d9b9 (refs/bisect/bad)
Author: Anthony Sottile <[email protected]>
Date: Sat Jan 12 20:05:13 2019 -0800
get-pip.py does now crash:
Python 3.8 is also affected.
get-pip.py comes from: https://bootstrap.pypa.io/get-pip.py
Reproducing depends if REALLOC() returns a new pointer, see below.
I attached get-pip2.py: simplified version which crash when using "./python get-pip2.py --help". If it doesn't crash, duplicate lines to make the file larger.
--
The root issue seems to be that tok->multi_line_start isn't updated on REALLOC() in tok_nextc():
I guess that multi_line_start should also be updated there?
vstinner commentedon Jul 16, 2019
PR 14433 seems to fix this issue and it contains a test.
My PR 14789 fixed get-pip.py, but it introduces another bug: I closed it.
ambv commentedon Jul 29, 2019
This is marked as release blocker but since BPO-37433 is still in review, I'm releasing 3.0b3 as is. Please resolve this by b4, I will block the last beta on this issue.
vstinner commentedon Jul 29, 2019
That sounds reasonable to me.
FYI I just approved (but not merged) PR 14433.
pablogsal commentedon Jul 29, 2019
I just merged PR 14433
vstinner commentedon Aug 21, 2019
This issue is a duplicate of bpo-37433 which has been fixed.