Opened 13 years ago

Closed 13 years ago

#3320 closed patch (fixed)

Delimited text plugin causes crash

Reported by: ccrook Owned by: gsherman
Priority: critical: causes crash or data corruption Milestone: Version 1.7.0
Component: Data Provider Version: Trunk
Keywords: delimited text provider plugin Cc:
Must Fix for Release: Yes Platform: All
Platform Version: Awaiting user input: no

Description

Following from message on qgis-user list at

http://lists.osgeo.org/pipermail/qgis-user/2010-December/010628.html

relating to QGis crash caused by delimited text plugin. This error appears to be due to reading files with variable number of fields, which is tested in the provider constructor function, but not in the nextFeature function.

The attached patch should fix this. It takes a slightly different approach to previously - instead of ignoring lines with the wrong number of fields, it adds blank strings to ensure that there are the expected number.

Also fixes a problem with line number counting.

Attachments (1)

delimited_text_crash_fix.patch (3.4 KB ) - added by ccrook 13 years ago.

Download all attachments as: .zip

Change History (3)

by ccrook, 13 years ago

comment:1 by gsherman, 13 years ago

Owner: changed from nobody to gsherman

Patch applied at r14890. Thanks!

comment:2 by gsherman, 13 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.