Bug report #15977

Crash when saving new rows in MSSQL layer with a join

Added by Martin Bain almost 4 years ago. Updated over 3 years ago.

Status:Closed
Priority:High
Assignee:-
Category:Data Provider/MSSQL
Affected QGIS version:2.18.0 Regression?:No
Operating System:Windows Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:Yes Copied to github as #:23893

Description

When editing an MSSQL layer which has a join defined I can edit and delete existing records but if I add one or more new records I get a minidump as soon as I attempt to save the edits.

If I remove the join I can save new records without error.

The format of the joined layer does not seem to matter. I still get a crash whether the joined layer is a table or view in SQL Server or a CSV file.

Using SQL Server 2008 R2 and QGIS 2.18.1

History

#1 Updated by Martin Bain almost 4 years ago

Meant to say the same joins work fine with a Spatialite layer

#2 Updated by Martin Bain almost 4 years ago

Just noticed a commit from 2 days ago: "Fix an array out of bounds access when committing data (#3807)" which may solve this issue. Will test and update

#3 Updated by Martin Bain almost 4 years ago

Tested with 2.19.0-2 and the problem still exists. However I notice the version of mssqlprovider.dll installed in qgis-dev is older (23/10/2016) than the one in 2.18 (26/11/2016). Maybe the nightly build doesn't include the commit from above.

#4 Updated by Jürgen Fischer almost 4 years ago

Martin Bain wrote:

Tested with 2.19.0-2 and the problem still exists. However I notice the version of mssqlprovider.dll installed in qgis-dev is older (23/10/2016) than the one in 2.18 (26/11/2016). Maybe the nightly build doesn't include the commit from above.

There are currently no nightlies of master for windows. The last qgis-dev build are from the late master_2 branch.

#5 Updated by Giovanni Manghi almost 4 years ago

  • Priority changed from Normal to High

#6 Updated by Giovanni Manghi over 3 years ago

  • Category set to Data Provider/MSSQL
  • Status changed from Open to Feedback

There are currently no nightlies of master for windows. The last qgis-dev build are from the late master_2 branch.

there are now.

#7 Updated by Giovanni Manghi over 3 years ago

  • Resolution set to fixed/implemented
  • Status changed from Feedback to Closed

I confirm the issue on 2.18.7 but is fixed on master.

Also available in: Atom PDF