Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GC bug in MOVED_PTS_TRANS (trans.c:2652/2662) #2319

Closed
ChrisJefferson opened this issue Mar 27, 2018 · 0 comments
Closed

GC bug in MOVED_PTS_TRANS (trans.c:2652/2662) #2319

ChrisJefferson opened this issue Mar 27, 2018 · 0 comments
Assignees
@james-d-mitchell
Labels
kind: bug: crash Issues describing bugs that cause GAP to crash, and PRs fixing them (used for release notes) kind: bug Issues describing general bugs, and PRs fixing them

Comments

@ChrisJefferson
Copy link
Contributor

In trans.c:2652, we have: AssPlist(out, ++len, INTOBJ_INT(i + 1));. This can cause a memory allocation, which breaks ptf2 (mentioned in the previous line). The same bug occurs with ptf4 a few lines later.
@james-d-mitchell james-d-mitchell mentioned this issue Mar 28, 2018
Merged
@fingolfin fingolfin added kind: bug Issues describing general bugs, and PRs fixing them kind: bug: crash Issues describing bugs that cause GAP to crash, and PRs fixing them (used for release notes) labels Mar 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@james-d-mitchell james-d-mitchell

Labels
kind: bug: crash Issues describing bugs that cause GAP to crash, and PRs fixing them (used for release notes) kind: bug Issues describing general bugs, and PRs fixing them
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fingolfin @ChrisJefferson @james-d-mitchell