ImHex icon indicating copy to clipboard operation
ImHex copied to clipboard

[Bug] Flatpak 1.18.2 crashes on open file in Fedora 36

Open meeas opened this issue 3 years ago • 11 comments

Operating System

Linux

What's the issue you encountered?

Note sure if you control the Flatpak build or if it is a third party.....finding this out is a major flatpak weakness. Please let me know if this is a third party build and I will try to hunt them down.

However it installs cleanly on Fedora 36 from flathub, and starts the GUI just fine. However when you open a file or try to create a new file, the app crashes with:

Gtk-Message: 01:31:34.030: Failed to load module "canberra-gtk-module"
Gtk-Message: 01:31:34.030: Failed to load module "pk-gtk-module"
Gtk-Message: 01:31:34.031: Failed to load module "canberra-gtk-module"
Gtk-Message: 01:31:34.031: Failed to load module "pk-gtk-module"
/usr/include/c++/11.3.0/bits/stl_vector.h:1045: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) [with _Tp = unsigned char; _Alloc = std::allocator<unsigned char>; std::vector<_Tp, _Alloc>::reference = unsigned char&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__n < this->size()' failed.

I'm guessing it is missing dependencies in the Flatpak build.

How can the issue be reproduced?

Just install from flathub, start app, and try opening a file. To see the error message, start from a commandline with flatpak run.

ImHex Version

1.18.2

ImHex Build Type

  • [ ] Nightly or built from sources

Additional context?

  • Additional information about your environment.
  • If possible and useful, please upload the binary you've been editing when the bug occured.

meeas avatar Jun 29 '22 07:06 meeas

Hey, could you try using the AppImage version ? (Please take it from the github actions builds, not the release)

iTrooz avatar Jun 29 '22 08:06 iTrooz

The AppImage opens the app fine, but when I click Open File is crashed before even showing the file picker, and errors with:

(imhex:61873): GLib-GIO-ERROR **: 02:02:05.004: Settings schema 'org.gnome.settings-daemon.plugins.xsettings' does not contain a key named 'antialiasing'
fish: Job 1, 'imhex.AppImage' terminated by signal SIGTRAP (Trace or breakpoint trap)

meeas avatar Jun 29 '22 08:06 meeas

That was running with Wayland. I also tried it with X11 and it threw Awaita Icon errors.

meeas avatar Jun 29 '22 08:06 meeas

Okay, what DE do you use on Fedora ?

iTrooz avatar Jun 29 '22 08:06 iTrooz

FlatPak ImHex works for me on Fedora 35 XFCE

iTrooz avatar Jul 02 '22 16:07 iTrooz

It seems that nativefiledialog-extended is the only reason for needing GTK. If you don't need to have a custom default path in the dialog, perhaps try NFD_PORTAL=ON in the CMake configure command so that portals are used instead of GTK dialogs? Especially since this is a build targetting Flatpak.

btzy avatar Jul 02 '22 17:07 btzy

WerWolf pushed a new version to flatpak. Could you try now ?

iTrooz avatar Jul 02 '22 22:07 iTrooz

I have not pushed a new flatpak release. I just fixed added the NFD_PORTAL=ON thing to the cmake script. There will be a new flatpak release in the next few days though when I release a new version of ImHex

WerWolv avatar Jul 02 '22 22:07 WerWolv

You could try the new native packages built for Fedora too: https://github.com/WerWolv/ImHex/actions/runs/2767434620#artifacts

jonathanspw avatar Jul 31 '22 00:07 jonathanspw

Note : We are waiting for Flatpak to use GCC 12 in their build environnments for the update to be made. (This should be sometime in August)

iTrooz avatar Aug 04 '22 17:08 iTrooz

Flatpak now has the latest version : https://flathub.org/apps/details/net.werwolv.ImHex Could you try with this one ?

iTrooz avatar Sep 11 '22 22:09 iTrooz

Just installed on Fedora 35 from FlatPak - I get the same issue the original report shows (Assertion '__n < this->size()' failed)

I don't see any Gnome messages in the log though

mich181189 avatar Sep 16 '22 12:09 mich181189

Sorry for the silence. I just tried the latest version in FlatHub version 1.23.0, same issue. I also tried your latest official Fedora package version 1.23.0 from the GitHub releases page.

Same issues. Both crash while trying to open a file. I'm running a full patched stable version of Fedora 36, with the default Gnome/Wayland DE.

From RPM build:

❯ imhex                                                                                                                                                                                                ─╯
[17:12:55] [INFO]  [main] Welcome to ImHex!
[17:12:55] [INFO]  [main] Native scaling set to: 1.0
/usr/include/c++/12/bits/stl_vector.h:1123: constexpr std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](size_type) [with _Tp = hex::prv::Provider*; _Alloc = std::allocator<hex::prv::Provider*>; reference = hex::prv::Provider*&; size_type = long unsigned int]: Assertion '__n < this->size()' failed.
[17:13:24] [FATAL] [main] Terminating with signal 6
fish: Job 1, 'imhex' terminated by signal SIGABRT (Abort)

From Flatpack build:

❯ flatpak run net.werwolv.ImHex                                                                                                                                                                        ─╯
[17:17:30] [INFO]  [main] Welcome to ImHex!
[17:17:30] [INFO]  [main] Native scaling set to: 1.0
/usr/include/c++/12.1.0/bits/stl_vector.h:1123: constexpr std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](size_type) [with _Tp = hex::prv::Provider*; _Alloc = std::allocator<hex::prv::Provider*>; reference = hex::prv::Provider*&; size_type = long unsigned int]: Assertion '__n < this->size()' failed.
[17:17:43] [FATAL] [main] Terminating with signal 6

meeas avatar Sep 16 '22 23:09 meeas

The official Fedora RPM build for 1.23 crashes on trying to open a file as well. It was reported at https://bodhi.fedoraproject.org/updates/FEDORA-2022-679b84ef3d

jonathanspw avatar Sep 17 '22 02:09 jonathanspw

Latest commit should address this once again, sorry.

WerWolv avatar Sep 17 '22 06:09 WerWolv

1.23.2 is out, should have fixed it :)

iTrooz avatar Sep 17 '22 17:09 iTrooz

Confirmed, fixed. Thank You!!!

meeas avatar Oct 02 '22 17:10 meeas