Project

General

Profile

Bug #10805

Darktable crash

Added by Jean-Luc Coulon about 4 years ago. Updated over 3 years ago.

Status:
Fixed
Priority:
Low
Category:
Masks
Target version:
Start date:
12/22/2015
Due date:
% Done:

100%

Estimated time:
Affected Version:
git development version
System:
all
bitness:
64-bit
hardware architecture:
amd64/x86

Description

Attached:
Backtrace
.cr2 file
.cr2.xmp file

darktable_bt_SS319X.txt (43.7 KB) darktable_bt_SS319X.txt Jean-Luc Coulon, 12/22/2015 07:23 PM
jlc_2566.cr2.xmp (18 KB) jlc_2566.cr2.xmp Jean-Luc Coulon, 12/22/2015 07:25 PM
jlc_2566.cr2 (23.5 MB) jlc_2566.cr2 Jean-Luc Coulon, 12/22/2015 07:28 PM

Associated revisions

Revision 8298829c (diff)
Added by Ulrich Pegelow about 4 years ago

masks (path, brush): use NAN as sentinel instead of magic number

the should partially fix #10805

Revision a14ca8f5 (diff)
Added by Ulrich Pegelow about 4 years ago

masks (path, brush): use NAN as sentinel instead of magic number

the should partially fix #10805

(cherry picked from commit 8298829cfbc41f8adbc6b07c0abee34a023a37bc)

History

#1 Updated by Roman Lebedev about 4 years ago

  • Assignee set to Aldric Renaudin
  • Category changed from General to Masks

Looks like masks.

#2 Updated by Ulrich Pegelow about 4 years ago

  • System changed from Debian to all
  • % Done changed from 0 to 10
  • Status changed from New to Confirmed

I don't get a crash but something is certainly broken here. In my case darktable even fails to read the mask shapes from the XMP file -> no masks in the impage after import. If I use the "load sidecar file" option 'darktable -d sql' reveals that no "insert into masks ..." statement is prepared.

#3 Updated by Ulrich Pegelow about 4 years ago

First things first, the non-reading of the masks from the XMP file seems to be caused by one of the shapes having an empty name: mask_name is set to <rdf:li/>.

Once I fixed it manually the original bug report is confirmed and everything wreaks havoc. darktable runs endlessly, eating up all available system memory and finally crashes.

#4 Updated by Ulrich Pegelow about 4 years ago

  • % Done changed from 10 to 50
  • Assignee changed from Aldric Renaudin to Ulrich Pegelow
  • Status changed from Confirmed to In Progress

#5 Updated by Ulrich Pegelow about 4 years ago

A tentative fix has been pushed to master. Please give it a try and check for potential regressions.

Remark: one of the path shapes in your sample image ("Chemin #13") escapes from being edited. Not sure why and no idea how to fix.

#6 Updated by Jean-Luc Coulon about 4 years ago

Well I was trying/testing everything.
The masks were in the circuit.

I remarked quite many strange things with the masks. For some things in the mask UI, I was replied the developper is no more active. So I supposed it was the same for all the things related to masks... And I left that...

I'm rebuilding dt. I will play more with the masks :)

#7 Updated by Ulrich Pegelow about 4 years ago

  • % Done changed from 50 to 100
  • Status changed from In Progress to Fixed

#8 Updated by Roman Lebedev over 3 years ago

  • Target version set to 2.2.0

Also available in: Atom PDF

Go to top