Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coredump on opening new tab but only with wayland #820

Closed
FrederikLauber opened this issue Apr 16, 2021 · 4 comments · Fixed by lxqt/qtermwidget#424
Closed

Coredump on opening new tab but only with wayland #820

FrederikLauber opened this issue Apr 16, 2021 · 4 comments · Fixed by lxqt/qtermwidget#424
Labels
bug platform:Wayland Wayland-specific issues

Comments

@FrederikLauber
Copy link

Hello,
experimenting with wayland and plasma atm.
I noticed that qterminal works but every time I want to open a new register, it coredumps.
This does not happen with X11+LXQT on the same machine, so my best guess is its an issue with wayland.

#####Steps to reproduce:
open qterminal
try to open a new register card either via the menu or by double clicking the register bar

#####Expected behaviour:
New open qterminal in register card

#####StackTrace:
Process 250842 (qterminal) of user 1000 dumped core. Stack trace of thread 250842: #0 0x00007f479affe54f n/a (libqtermwidget5.so.0 + 0x4754f) #1 0x00007f479b00b08a n/a (libqtermwidget5.so.0 + 0x5408a) #2 0x00007f479a8f10cf _ZN7QWidget5eventEP6QEvent (libQt5Widgets.so.5 + 0x19c0cf) #3 0x00007f479a8af762 _ZN19QApplicationPrivate13notify_helperEP7QObjectP6QEvent (libQt5Widgets.so.5 + 0x15a762) #4 0x00007f4799de363a _ZN16QCoreApplication15notifyInternal2EP7QObjectP6QEvent (libQt5Core.so.5 + 0x2b663a) #5 0x00007f479670d9cc n/a (libQt5WaylandClient.so.5 + 0x859cc) #6 0x00007f479a1b46d9 _ZN22QGuiApplicationPrivate20_q_updateFocusObjectEP7QObject (libQt5Gui.so.5 + 0x12e6d9) #7 0x00007f4799e1a9d0 n/a (libQt5Core.so.5 + 0x2ed9d0) #8 0x00007f479a1c4273 _ZN7QWindow18focusObjectChangedEP7QObject (libQt5Gui.so.5 + 0x13e273) #9 0x00007f479a8ea7d3 _ZN7QWidget8setFocusEN2Qt11FocusReasonE (libQt5Widgets.so.5 + 0x1957d3) #10 0x00007f479aff11b2 _ZN11QTermWidget4initEi (libqtermwidget5.so.0 + 0x3a1b2) #11 0x0000560a260ca571 n/a (qterminal + 0x31571) #12 0x0000560a260cac19 n/a (qterminal + 0x31c19) #13 0x0000560a260cf417 n/a (qterminal + 0x36417) #14 0x0000560a260cf9ee n/a (qterminal + 0x369ee) #15 0x0000560a260c70de n/a (qterminal + 0x2e0de) #16 0x0000560a260ba559 n/a (qterminal + 0x21559) #17 0x00007f4799e1a9d0 n/a (libQt5Core.so.5 + 0x2ed9d0) #18 0x00007f479a8a8f73 _ZN7QAction9triggeredEb (libQt5Widgets.so.5 + 0x153f73) #19 0x00007f479a8ab855 _ZN7QAction8activateENS_11ActionEventE (libQt5Widgets.so.5 + 0x156855) #20 0x00007f479aa2d52b n/a (libQt5Widgets.so.5 + 0x2d852b) #21 0x00007f479aa34c62 n/a (libQt5Widgets.so.5 + 0x2dfc62)

System Information
  • Distribution & Version: Arch Linux (fully updated)
  • Kernel: 5.11.14-arch1-1
  • Qt Version: 5.15.2-1
  • qtermwidget Version: 0.16.1-3
  • qterminal: qterminal 0.16.1-1
@tsujan
Copy link
Member

tsujan commented Apr 16, 2021

What's "register card"? You probably mean #806

@tsujan tsujan closed this as completed Apr 16, 2021
@FrederikLauber
Copy link
Author

I guess it is the same underlying issue but I am not talking about splitting the existing console, I speak about what happens if you CTRL+Shift+T, in the German version that is called "Neue Registerkarte". I guess my English translation was incorrect, is it maybe "New Tab"?

I attached a screenshot (here from the X11 version but the menu point is the same)
2

@tsujan tsujan reopened this Apr 16, 2021
@tsujan tsujan changed the title Coredump opening new register card but only with wayland Coredump on opening new tab but only with wayland Apr 16, 2021
@tsujan
Copy link
Member

tsujan commented Apr 16, 2021

@FrederikLauber
Thanks for the screenshot! Yes, it's "New Tab". Your report is opened to be investigated.

@tsujan
Copy link
Member

tsujan commented Apr 20, 2021

I also saw this crash under Plasma Wayland, with a backtrace that was the same as in #806. But let's keep this open for the sake of testing.

The method PlainTextDecoder::decodeLine() seems very suspicious to me; I didn't find any guarantee that characters[i] exists.

EDIT: See #806

@tsujan tsujan added the bug label Apr 20, 2021
tsujan added a commit to lxqt/qtermwidget that referenced this issue May 2, 2021
With this check, I can't make QTerminal crash by splitting or opening new tabs under Plasma Wayland.

Fixes lxqt/qterminal#806 and fixes lxqt/qterminal#820
yan12125 pushed a commit to lxqt/qtermwidget that referenced this issue May 2, 2021
With this check, I can't make QTerminal crash by splitting or opening new tabs under Plasma Wayland.

Fixes lxqt/qterminal#806 and fixes lxqt/qterminal#820
@yan12125 yan12125 added the platform:Wayland Wayland-specific issues label Jan 27, 2023
@yan12125 yan12125 moved this to Done in Wayland Support Jul 26, 2023
inventor2525 pushed a commit to inventor2525/qtermwidget that referenced this issue May 14, 2024
With this check, I can't make QTerminal crash by splitting or opening new tabs under Plasma Wayland.

Fixes lxqt/qterminal#806 and fixes lxqt/qterminal#820
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug platform:Wayland Wayland-specific issues
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants