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.

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

Multiple projectors with SPECTUBMatrix cause double free error #1183

Closed
samdporter opened this issue Apr 17, 2023 · 3 comments · Fixed by #1169
Closed

Multiple projectors with SPECTUBMatrix cause double free error #1183

samdporter opened this issue Apr 17, 2023 · 3 comments · Fixed by #1169

Comments

@samdporter
Copy link
Contributor

samdporter commented Apr 17, 2023

I think there's an error in freeing memory in the destructor for the ProjectorByBinPairUsingProjMatrixByBin class when used with the SPECTUB class

I'll include some minimal examples as well as valgrind output below.

Created a new issue here as opposed to SIRF issue #1101 as it's definitely a STIR issue.

I've tried recompiling SIRF/STIR without OpenMP as suggested in the issue above, but it hasn't made any difference.

@KrisThielemans
Copy link
Collaborator

Probably best to wait till #1169 is merged. That will change things.

@samdporter
Copy link
Contributor Author

samdporter commented Apr 17, 2023

This repo now contains some minimal SIRF & STIR scripts that throw the error
double free or corruption (!prev) Aborted (core dumped)
The errors occur on exiting the script after setting up two acquisition models with SPECTUB matrices

The repo also contains some valgrind output files the I'll copy here:
for SIRF
for STIR

@KrisThielemans KrisThielemans linked a pull request Apr 18, 2023 that will close this issue
@samdporter
Copy link
Contributor Author

Appears fixed by #1169

see updated repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants