Comment 9 for bug 732693

Revision history for this message
ron&bram (ronaldestherbram) wrote :

You missed the point.
The trace line just writes the length of prevrouteids.
With the plan in question, the length of prevrouteids can exceed the defined size (4096) of the char array in the StrOp.fmt call. This causes the crash.
The fix I made is to only add more route ID's to the routeids of a track when the length of prevrouteids does not exceed 4000:

if( !isInList && doIt && StrOp.len(prevrouteids) < 4000 ) {

I suggested to comment this fix, but first to remove the trace line, because it was for debugging only. Commit?