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

[Bug] v.random segfaults due to access of unallocated memory #1024

Closed
marisn opened this issue Oct 16, 2020 · 1 comment
Closed

[Bug] v.random segfaults due to access of unallocated memory #1024

marisn opened this issue Oct 16, 2020 · 1 comment
Labels
bug Something isn't working
Milestone
7.8.5

Comments

@marisn
Copy link
Contributor

marisn commented Oct 16, 2020

v.random output=test_points npoints=6 restrict=zipcodes_wake column=Depth --v --o
segfault at main.c:673

Reported by Eric Patton in GRASS-user.
@marisn marisn added the bug Something isn't working label Oct 16, 2020
marisn pushed a commit to marisn/grass that referenced this issue Oct 16, 2020
v.random: Fix segfault as per bug OSGeo#1024
f73b6cd 
cats_array memory is allocated only when fields for parm.input->answer are > 0
@marisn marisn changed the title [Bug] v.random Vect_cat_get needs guard for case when cats are not found [Bug] v.random segfaults due to access of unallocated memory Oct 16, 2020
neteler pushed a commit that referenced this issue Oct 26, 2020
@marisn
v.random: Fix segfault as per bug #1024 (#1025)
c3ad638 
cats_array memory is allocated only when fields for parm.input->answer are > 0

Co-authored-by: Māris Nartišs <maris.nartiss@lu.lv>
neteler pushed a commit that referenced this issue Oct 26, 2020
@marisn @neteler
v.random: Fix segfault as per bug #1024 (#1025)
3b6d257 
cats_array memory is allocated only when fields for parm.input->answer are > 0

Co-authored-by: Māris Nartišs <maris.nartiss@lu.lv>
@neteler
Copy link
Member

neteler commented Nov 21, 2020

Since #1025 was merged and backported, can this be closed?

v.random output=test_points npoints=6 restrict=zipcodes_wake column=Depth --v --o

works now fine.

@neteler neteler added this to the 7.8.5 milestone Nov 21, 2020
@neteler neteler closed this as completed Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
7.8.5
Development

No branches or pull requests
2 participants
@neteler @marisn