[GRASS-dev] Re: [GRASS GIS] #1650: v.net.salesman - add sequence output

[GRASS GIS]

#1650: v.net.salesman - add sequence output
-------------------------+--------------------------------------------------
 Reporter:  gfleming     |       Owner:  grass-dev@…              
     Type:  enhancement  |      Status:  new                      
 Priority:  normal       |   Milestone:  6.4.3                    
Component:  Default      |     Version:  unspecified              
 Keywords:               |    Platform:  Unspecified              
      Cpu:  Unspecified  |  
-------------------------+--------------------------------------------------

Comment(by mmetz):

 Replying to [comment:2 mlennert]:
 > No idea if it is linked to your commit, but I get a segfault running the
 following on a fresh checkout (r51586)  & compile of trunk (including a
 'make distclean'):

 {{{
 > v.net in=streets_wake points=schools_wake out=streets_with_schools
 operation=connect thresh=50
 > v.net.salesman in=streets_with_schools out=salesroute ccats=1-999
 }}}

 > I've attached the backtrace.
 >
 > If it's not linked, I can open a separate ticket.

 I can not reproduce it on Linux 64 bit, but I suspect buffer overflow in
 buf and buf2 according to the backtrace, see also my recently added
 comment in the source code [0]. I will change this verbose message to a
 debug message and make it print out one arc / node per line, that should
 avoid the buffer overflow. The graph in the example has 994 arcs, but buf
 can only hold a bit less than 300 arcs with 5-digit category numbers.

 Markus M

 [0]
 https://trac.osgeo.org/grass/browser/grass/trunk/vector/v.net.salesman/main.c#L459

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1650#comment:3>
GRASS GIS <http://grass.osgeo.org>