Opened 18 years ago

Closed 8 years ago

#1209 closed defect (fixed)

incorrect MapInfo style string causes system exception

Reported by: richard.matsunaga@… Owned by: warmerdam
Priority: normal Milestone:
Component: OGR_SF Version: unspecified
Severity: normal Keywords: mitab
Cc:

Description (last modified by Jukka Rahkonen)

Upon creating a new feature on a MapInfo layer, if the style string in the
feature is invalid, the MapInfo driver will throw an exception instead of
reporting an error.

For example, the following string is only invalid by an extra space character
after the comma:

"SYMBOL(c:#000000, id:\"ogr-sym-1\")"

and will cause a crash on the OGR_L_CreateFeature() call.

Attachments (1)

ogr_style_error.jml (1.3 KB ) - added by Jukka Rahkonen 9 years ago.

Download all attachments as: .zip

Change History (8)

by Jukka Rahkonen, 9 years ago

Attachment: ogr_style_error.jml added

comment:1 by Jukka Rahkonen, 9 years ago

Description: modified (diff)

I can reproduce the error with attached OpenJUMP file which has a faulty style string

SIMBOL(c:#000000, id:\"ogr-sym-1\")

System tolerates extra spaces nowadays so I had to introduce a bigger error.

Command "ogr2ogr -f "mapinfo file" ogrstyle.mif ogr_style_error.jml" makes ogr2ogr.exe to crash on Windows 7 without any error messages.

Tested with GDAL 2.0-dev

comment:2 by Jukka Rahkonen, 9 years ago

Keywords: mitab added

comment:3 by Even Rouault, 9 years ago

trunk r28313, branches/1.11 r28314 "MITAB: fix segfault in CreateFeature() if passing an invalid OGR feature style string (#1209)"

comment:4 by Jukka Rahkonen, 9 years ago

If this is advertized as fixed in http://trac.osgeo.org/gdal/wiki/Release/1.11.2-News?version=1, is the ticket mature enough for closing?

in reply to:  4 comment:5 by Even Rouault, 9 years ago

Replying to jratike80:

If this is advertized as fixed in http://trac.osgeo.org/gdal/wiki/Release/1.11.2-News?version=1, is the ticket mature enough for closing?

This is just because it is is a mitab issue.

comment:6 by Even Rouault, 9 years ago

Priority: highnormal

All those tickets have more than one year and nobody has acted on it, so the priority is not so high

comment:7 by Even Rouault, 8 years ago

Resolution: fixed
Status: newclosed

Batch closing of MITAB tickets fixed in GDAL. They were kept open because not merged into MITAB separate repository, but the latter one is inactive, so let's close them definitely.

Note: See TracTickets for help on using tickets.