aurelienpierreeng / ansel

A darktable fork minus the bloat plus some design vision.
https://ansel.photos
GNU General Public License v3.0
688 stars 22 forks source link

Program crashed after changing orientation of cropping and clicking "Apply" #275

Closed AlynxZhou closed 11 months ago

AlynxZhou commented 11 months ago

Description of the bug

Just like the title.

To Reproduce

  1. Go to darkroom.
  2. Click on crop module.
  3. Click "Edit".
  4. Click the arrow button next to ratio to change orientation.
  5. Click "Apply"

Expected behavior

Photo is cropped.

Context

Screenshots

Screencast

This is the CLI output, it looks like some thread locks error:

ansel-crop-crash.txt

The program says it writes backtrace to /tmp, but the file is empty, so I assume if I start it from CLI it will just output to terminal.

Which commit introduced the error

I am using https://github.com/aurelienpierreeng/ansel/commit/0a2bf45eff53d73e489e4fdd389812d413e3efc2.

The last commit I tried without this issue (but the crop orientation still doesn't work correctly) is https://github.com/aurelienpierreeng/ansel/commit/3cf785d378a9404ca894691dd79a96030eec267f.

System

Additional context

AlynxZhou commented 11 months ago

ansel-crop-crash-debug.txt

I get GDB output of a debug build. Not sure why backtrace is not written to the backtrace file in /tmp for me, but that's not important.

aurelienpierre commented 11 months ago

Fixed by 53c609c

AlynxZhou commented 11 months ago

Thanks!