ogr2ogr crashes when converting SHP to MIF in version 1.70

[rhodges]

You should have no problem reproducing this, as the problem is a null pointer dereference on the return value of GetLayerDefn() from a newly created MIF layer. It appears that the MIF driver behaves differently than others in that it doesn't create this layer definition until you first start modifying it; unfortunately the new ogr2ogr code assumes that it will have done. I will attach a diff showing one way to defend against this, but perhaps you would prefer the approach of modifying the MIF driver instead -- I don't have enough familiarity with the code to feel confident attacking it from that angle.

[rhodges]

Diff showing one possible fix for this crash

[warmerdam]

Chaitanya,

Could you look into this? You will need to cooperate with Daniel on changes that need to go upstream into mitab.

[Even Rouault]

rhodges,

I've applied your patch in trunk (r18749) and in branches/1.7 (r18750) as a quick regression fix for it to go in time for 1.7.1. Tests added in r18751. Thanks !

Chaitanya,

I've opened ticket #3391 to track the fact that the MapInfo driver doesn't behave like other drivers.