Infinite loop in osm2navit

[hgavin]

If you pass in some osm-format data which defines fewer than 4 nodes and then a way, osm2navit enters an infinite loop. E.g.

<?xml version="1.0" encoding="UTF-8"?>
<osm version="0.5">
  <node id="1" lat="54.46355" lon="-8.189"/>
  <node id="2" lat="54.46439" lon="-8.18838"/>
  <node id="3" lat="54.46495" lon="-8.18808"/>
  <way id="4">
    <nd ref="1"/>
    <nd ref="2"/>
    <nd ref="3"/>
    <tag k="highway" v="road"/>
  </way>
</osm>

This is due to the line "int interval=count/4;" in node_item_get(). If the node count is less than 4, interval gets rounded to 0, and so the index "p" is never incremented or decremented and the loop never exits.

A simple patch is attached which simply sets interval to 1 should it be 0.

[hgavin]

Fix for infinite loop in osm2navit.c

[horwitz]

The patch has been applied to revision 2381.