Opened 18 years ago
Closed 9 years ago
#1209 closed defect (fixed)
incorrect MapInfo style string causes system exception
Reported by: | Owned by: | warmerdam | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | OGR_SF | Version: | unspecified |
Severity: | normal | Keywords: | mitab |
Cc: |
Description (last modified by )
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)
Change History (8)
by , 10 years ago
Attachment: | ogr_style_error.jml added |
---|
comment:1 by , 10 years ago
Description: | modified (diff) |
---|
comment:2 by , 10 years ago
Keywords: | mitab added |
---|
comment:3 by , 10 years ago
trunk r28313, branches/1.11 r28314 "MITAB: fix segfault in CreateFeature() if passing an invalid OGR feature style string (#1209)"
follow-up: 5 comment:4 by , 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?
comment:5 by , 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 , 9 years ago
Priority: | high → normal |
---|
All those tickets have more than one year and nobody has acted on it, so the priority is not so high
comment:7 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
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.
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