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

grdblen crashes with externals when input grids must be resampled. #5715

Closed
joa-quim opened this issue Aug 30, 2021 · 3 comments · Fixed by #8406
Closed

grdblen crashes with externals when input grids must be resampled. #5715

joa-quim opened this issue Aug 30, 2021 · 3 comments · Fixed by #8406
Labels
bug Something isn't working

Comments

@joa-quim
Copy link
Member

See topic in forum

The problem arises when grdsample is called and the output tries to overwrite an external object
https://github.com/GenericMappingTools/gmt/blob/master/src/grdblend.c#L474

B[n].G->data then points to invalid memory and a crash follows later when trying to access it.

@joa-quim joa-quim added the bug Something isn't working label Aug 30, 2021
@PaulWessel
Copy link
Member

That grdsample command writes to a temporary file called grdblend_resampled_?_?.grd, so I dont think related to writing. Perhaps grdblend is trying to do something to the external input grid related to pads?

@PaulWessel
Copy link
Member

I should have some time this afternoon to try out the Julia debug instructions etc.

@joa-quim
Copy link
Member Author

Debugging from Julia calls is becoming impossibly slow. Probably I meant what happens in
https://github.com/GenericMappingTools/gmt/blob/master/src/grdblend.c#L501

joa-quim added a commit that referenced this issue Mar 16, 2024
Fixes #5715 but this unveiled other bugs in grdblend that will be addressed later.
joa-quim added a commit that referenced this issue Mar 16, 2024
…8406)

Fixes #5715 but this unveiled other bugs in grdblend that will be addressed later.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants