Opened 10 years ago

Closed 10 years ago

#697 closed defect (fixed)

v.surf.rst memory problems

Reported by: fpachecouy Owned by: grass-dev@…
Priority: normal Milestone: 6.4.0
Component: Default Version: svn-develbranch6
Keywords: Cc:
CPU: Unspecified Platform: Unspecified

Description

On Ubuntu 9.04, GRASS 6.4 (from deb packeage) and GRASS 6.5 (svn).

When I run v.surf.rst with valid parameters program crash with this output: v.surf.rst tension=80.0 input=curvas elev=curvasMDT scolumn='cota' segmax=100 dmin=50 Percent complete: Reading lines from vector map ... Reading nodes from vector map ...

100% 125%

* glibc detected * v.surf.rst: munmap_chunk(): invalid pointer: 0xb7f85e60 *

Output (using valgrind): valgrind v.surf.rst tension=80.0 input=curvas elev=curvasMDT scolumn='cota' segmax=100 dmin=50

==16812== Memcheck, a memory error detector.

==16812== Copyright (C) 2002-2008, and GNU GPL'd, by Julian Seward et al.

==16812== Using LibVEX rev 1884, a library for dynamic binary translation.

==16812== Copyright (C) 2004-2008, and GNU GPL'd, by OpenWorks? LLP.

==16812== Using valgrind-3.4.1-Debian, a dynamic binary instrumentation framework.

==16812== Copyright (C) 2000-2008, and GNU GPL'd, by Julian Seward et al.

==16812== For more details, rerun with: -v

==16812==

Percent complete:

Reading lines from vector map ... Reading nodes from vector map ...

100% 125%

==16812== Conditional jump or move depends on uninitialised value(s)

==16812== at 0x4055D5D: db_CatValArray_free (value.c:350)

==16812== by 0x403FF69: IL_vector_input_data_2d (vinput2d.c:305)

==16812== by 0x804C141: main (main.c:692)

==16812==

==16812== Conditional jump or move depends on uninitialised value(s)

==16812== at 0x4055D62: db_CatValArray_free (value.c:350)

==16812== by 0x403FF69: IL_vector_input_data_2d (vinput2d.c:305)

==16812== by 0x804C141: main (main.c:692)

==16812==

==16812== Conditional jump or move depends on uninitialised value(s)

==16812== at 0x4025DBD: free (vg_replace_malloc.c:323)

==16812== by 0x4055D71: db_CatValArray_free (value.c:364)

==16812== by 0x403FF69: IL_vector_input_data_2d (vinput2d.c:305)

==16812== by 0x804C141: main (main.c:692)

==16812==

==16812== Invalid free() / delete / delete[]

==16812== at 0x4025DFA: free (vg_replace_malloc.c:323)

==16812== by 0x4055D71: db_CatValArray_free (value.c:364)

==16812== by 0x403FF69: IL_vector_input_data_2d (vinput2d.c:305)

==16812== by 0x804C141: main (main.c:692)

==16812== Address 0x4034e60 is not stack'd, malloc'd or (recently) free'd

WARNING: strip exists with insufficient data

Thanks. Fernando.

Change History (3)

comment:1 Changed 10 years ago by helena

Priority: majornormal

Fernando - you are missing zcolumn for the values to be interpolated but you are defining scolumn for the smoothing values. I will change the priority to normal - the code should recognize the user mistake and let him know that zcol is missing. If your data are 3D and the value to interpolate is the z-coordinate then you need to define layer=0. I have the code modification with -z flag instead of layer=0 for this case, I just did not have time to submit.

comment:2 Changed 10 years ago by fpachecouy

Great. It was my mistake. Sorry.

I will try to use the 3D option. Thank you very much.

comment:3 Changed 10 years ago by helena

Resolution: fixed
Status: newclosed
Version: unspecifiedsvn-develbranch6
Note: See TracTickets for help on using tickets.