PhotoFlare / photoflare

Quick, simple but powerful Cross Platform image editor.
https://photoflare.io
GNU General Public License v3.0
402 stars 69 forks source link

Crashes changing contrast with Magick: abort due to signal 6 (SIGABRT) #453

Open ryanrapini opened 3 years ago

ryanrapini commented 3 years ago

Describe the bug Clicking any button to alter contrast of the attached image will crash PhotoFlare.

To Reproduce Steps to reproduce the behavior:

  1. Open Photoflare
  2. Open this image
  3. Click "Contrast (-)"
  4. Photoflare crashes completely.

Desktop:

Additional context

[ryan@guest-pc-manjaro laradock]$ photoflare 
QMetaObject::connectSlotsByName: No matching signal for on_image_filtered(QImage)
QMetaObject::connectSlotsByName: No matching signal for on_TextTool_finished()
QMetaObject::connectSlotsByName: No matching signal for on_show_hotspots_triggered()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
QDockWidgetLayout::addItem(): please use QDockWidgetLayout::setWidget()
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
libpng warning: known incorrect sRGB profile
libpng warning: profile matches sRGB but writing iCCP instead
terminate called after throwing an instance of 'Magick::WarningCoder'
  what():  Magick: profile matches sRGB but writing iCCP instead () reported by coders/png.c:1105 (PNGWarningHandler)
Magick: abort due to signal 6 (SIGABRT) "Abort"...
Aborted (core dumped)
ryanrapini commented 3 years ago

Can attach coredump if needed

DylanC commented 3 years ago

@ryanrapini - Confirmed. Seems to be caused by the colour profile data. I think maybe we should use mogrify in graphicsmagick to get around the problem.