Bug report #15891

Crash when the first SVG path is empty

Added by Mike Blechschmidt almost 3 years ago. Updated over 2 years ago.

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

Description

If the first path in
"Path(s) to search for Scalable Verctor Graphic (SVG) symbols"
is empty, the selection of an SVG element is crashing in QGIS 2.18.
Older versions can handle this.

Unbenannt.png (55.3 KB) Mike Blechschmidt, 2017-01-06 02:04 AM

Associated revisions

Revision 6f759a87
Added by Alexander Bruy over 2 years ago

ignore empty SVG path (fix #15891)

Revision fe7d49a0
Added by Alexander Bruy over 2 years ago

Merge pull request #4470 from alexbruy/ignore-empty-svg-path

ignore empty SVG path (fix #15891)

Revision 0780756f
Added by Alexander Bruy over 2 years ago

ignore empty SVG path (fix #15891)

(cherry picked from commit 6f759a87c9beda8b823ff107fdde680f3de9f720)

History

#1 Updated by Giovanni Manghi over 2 years ago

  • Crashes QGIS or corrupts data changed from No to Yes
  • Status changed from Open to Feedback

Cannot confirm on 2.18.2, on both Linux and Windows. No crash.
Please add more details.

#2 Updated by Mike Blechschmidt over 2 years ago

I have tested again with Windows and 2.18.2. It crashes.
See also https://lists.fossgis.de/pipermail/fossgis-talk-liste/2016-November/008228.html (german)

#3 Updated by Giovanni Manghi over 2 years ago

  • Priority changed from High to Severe/Regression
  • Status changed from Feedback to Open
  • Affected QGIS version changed from 2.18.0 to 2.18.2

Confirmed now. Does not happen on LTR.

#4 Updated by Giovanni Manghi over 2 years ago

  • Target version set to Version 2.18
  • Affected QGIS version changed from 2.18.2 to 2.18.4
  • Assignee deleted (Giovanni Manghi)

2.18.4 also affected.

#5 Updated by Giovanni Manghi over 2 years ago

  • Regression? set to Yes

#6 Updated by Giovanni Manghi over 2 years ago

  • Priority changed from Severe/Regression to High

#7 Updated by Giovanni Manghi over 2 years ago

  • Easy fix? set to No

#8 Updated by Alexander Bruy over 2 years ago

  • Pull Request or Patch supplied changed from No to Yes
  • Description updated (diff)

#9 Updated by Alexander Bruy over 2 years ago

  • % Done changed from 0 to 100
  • Status changed from Open to Closed

#10 Updated by Giovanni Manghi over 2 years ago

  • Resolution set to fixed/implemented

Also available in: Atom PDF