Skip to content

Conversation

@fingolfin
Copy link
Member

The call to NEW_PLIST can cause a garbage collection, which would render the seen pointer returned by ResizeInitTmpTrans invalid.

Also simplify some similar (but not buggy) code in FuncIMAGE_SET_TRANS_INT

Fixes #2333

The call to `NEW_PLIST` can cause a garbage collection, which would
render the `seen` pointer returned by `ResizeInitTmpTrans` invalid.

Also simplify some similar (but not buggy) code in
`FuncIMAGE_SET_TRANS_INT`

Fixes gap-system#2333
@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 Apr 11, 2018
@codecov
Copy link

codecov bot commented Apr 11, 2018

Codecov Report

Merging #2353 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2353      +/-   ##
==========================================
+ Coverage   73.13%   73.13%   +<.01%     
==========================================
  Files         482      482              
  Lines      246496   246495       -1     
==========================================
+ Hits       180265   180266       +1     
+ Misses      66231    66229       -2
Impacted Files Coverage Δ
src/trans.c 99.81% <100%> (-0.01%) ⬇️
src/hpc/threadapi.c 37.55% <0%> (ø) ⬆️
src/funcs.c 80.47% <0%> (+0.27%) ⬆️

@fingolfin fingolfin merged commit f6d58bc into gap-system:master Apr 12, 2018
@fingolfin fingolfin deleted the mh/fix-2333 branch April 12, 2018 21:22
@fingolfin fingolfin added the release notes: added PRs introducing changes that have since been mentioned in the release notes label Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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 release notes: added PRs introducing changes that have since been mentioned in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants