Bug #9520

crash in lowlight module during export due to eq producing NaNs

Added by Igor Kuzmin almost 7 years ago. Updated about 6 years ago.

Target version:
Start date:
Due date:
% Done:


Estimated time:
Affected Version:
git development version
hardware architecture:


I've come across one specific combination of image+history stack that produces segfault in lowlight.c during HQ export on darktable 1.2.2 (tested both release and current HEAD of darktable-1.2.x branch).
raw file:
minimized history stack that still produce a crash:

darktable_bt_258E0W.txt (48.2 KB) darktable_bt_258E0W.txt backtrace Igor Kuzmin, 07/18/2013 09:06 AM
darktable_bt_ZF96ZW.txt (41.4 KB) darktable_bt_ZF96ZW.txt backtrace with -O0 and no OpenMP Igor Kuzmin, 07/18/2013 01:12 PM


#1 Updated by Igor Kuzmin almost 7 years ago

  • System changed from other GNU/Linux to all
  • % Done changed from 0 to 10
  • Status changed from New to Confirmed

Just tested on my Mac with 1.2.2 release DMG, I get the same crash, so it's not specific to my Linux system. hanatos tried it yesterday on master - it didn't crash, so the bug is most likely only present in 1.2.x branch and was fixed in master. Some cherry-picking would be nice...

#2 Updated by Igor Kuzmin almost 7 years ago

hmm, what's strange though is that I just checked and there's no code difference in lowlight.c between master and darktable-1.2.x branches. maybe hanatos just did non-fullres non-HQ export (it doesn't crash in that case)...

#3 Updated by Igor Kuzmin almost 7 years ago

I've recompiled with -O0 and without OpenMP. Crash is still there, I've attached the backtrace.

#4 Updated by Igor Kuzmin almost 7 years ago

  • Subject changed from crash in lowlight module during export to crash in lowlight module during export due to eq producing NaNs

just to keep record: equalizer module (atrous) was identified as an original source of the problem - it produces NaNs

#5 Updated by Tobias Ellinghaus over 6 years ago

  • % Done changed from 10 to 20
  • Status changed from Confirmed to Incomplete

Can you still confirm this with git master?

#6 Updated by Igor Kuzmin over 6 years ago

I remember asking hanatos about this bug - he said he did come up with a fix, but it wasn't good enough for him to be satisfied so the patch wasn't committed to git. I don't remember if it applies to git master.

#7 Updated by Roman Lebedev about 6 years ago

  • bitness set to 64-bit
  • Affected Version changed from 1.2.2 to git development version
  • % Done changed from 20 to 10
  • Status changed from Incomplete to Confirmed

Still crashes with current git master

process._omp_fn.0 () at /home/lebedevri/darktable/src/iop/lowlight.c:120
120 return lut[bin1]*f + lut[bin0]*(1.-f);

Also available in: Atom PDF

Go to top