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

Fix allocation of temporary buffer in `Wcsprm.set`. #2350

Merged
merged 1 commit into from Apr 22, 2014

Conversation

@mdboom
Copy link
Contributor

mdboom commented Apr 21, 2014

The number of axes returned may be up to twice as many as requested. Rather than being smart about it, just allocate the buffer to be twice as large -- these are very small buffers in any case (one int per dimension).

@mdboom mdboom added the wcs label Apr 21, 2014
returned may be up to twice as many as requested.  Rather than being
smart about it, just allocate the buffer to be twice as large -- these
are very small buffers in any case (one int per dimension).
mdboom added a commit that referenced this pull request Apr 22, 2014
Fix allocation of temporary buffer in `Wcsprm.set`.
@mdboom mdboom merged commit 0946ce2 into astropy:master Apr 22, 2014
mdboom added a commit that referenced this pull request Apr 23, 2014
Fix allocation of temporary buffer in `Wcsprm.set`.
@mdboom mdboom deleted the mdboom:wcs/invalid-read branch May 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.