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

Segfault when issuing multiple window commands [GTK] #112

Closed
PannenetsF opened this issue May 6, 2020 · 12 comments
Closed

Segfault when issuing multiple window commands [GTK] #112

PannenetsF opened this issue May 6, 2020 · 12 comments
Assignees
Labels
Bug Something isn't working Platform: Linux

Comments

@PannenetsF
Copy link

Iust got the newest release.
When a imported my new account and "Save all settings" it crashed.
OS: Ubuntu 18.04.04
WM: i3
Here is the last lines output.

[INFO] 127.0.0.1 - - [06/May/2020 14:25:36] "PUT /api/settings HTTP/1.1" 200 -
[INFO] 127.0.0.1 - - [06/May/2020 14:25:36] "GET /?window_id=fa52675f-e2c0-4551-a19e-9cf77e992e1f HTTP/1.1" 200 -
[INFO] 127.0.0.1 - - [06/May/2020 14:25:36] "GET /close-window?window_id=33fa7290-2d1c-416e-aaf1-17c40b69027e HTTP/1.1" 204 -
[INFO] 127.0.0.1 - - [06/May/2020 14:25:36] "GET /static/dist/1.2005020754/main.js HTTP/1.1" 200 -
[INFO] Logged in as [email protected]
[INFO] 127.0.0.1 - - [06/May/2020 14:25:36] "GET /api/settings HTTP/1.1" 200 -
[1]    9840 abort (core dumped)  Kanmail
@Fizzadar Fizzadar added Bug Something isn't working Platform: Linux labels May 6, 2020
@Fizzadar
Copy link
Member

Fizzadar commented May 6, 2020

@PannenetsF do the settings get updated when this happens?

@PannenetsF
Copy link
Author

@PannenetsF do the settings get updated when this happens?

Well, it remains the same as before.

And after some attempts, I found there are about a 70% chance of that happening.

@Fizzadar
Copy link
Member

Have replicated this on Ubuntu 20 test machine; seems like calling JS reload on the main window is prone to crashes. Unfortunately no logs or anything indicating why even in dev mode.

Going to try to identify if it's just injecting reload into the main window or reloads in general.

@Fizzadar
Copy link
Member

Update: my theory is that calling reload from the server crashes because the page reloads before the return value can be passed @ https://github.com/Oxygem/Kanmail/blob/master/kanmail/window.py#L67. Seems specifically to affect GTK but makes sense.

@PannenetsF
Copy link
Author

Part of this bug has been fixed in the newest version!

But there's something left: If you keep the original setting and save it, the bug is going on. But if any changes are made, it turns to ok!

Thank you!

@Fizzadar
Copy link
Member

Interesting! Sadly in my Ubuntu 20 VM it still happens whether changing or not.

But I'm beginning to think this is an issue resetting the account instances on the server side, rather than anything related to GTK/reloading. Continuing investigation!

@Fizzadar
Copy link
Member

More info - can be replicated consistently by opening the settings/license/contacts windows over and over, will segfault after a a few in a row. Running with gdb points towards something in libwebkit2gtk.

Next up I need to get into the pywebview code and see if a GTK call there is causing this somehow.

Good news is this confirms it's nothing to do with settings/account resetting :)

@Fizzadar Fizzadar changed the title Unexpected Core Dumped while click "Save all settings" Segfault when issuing multiple window commands [GTK] May 24, 2020
@Fizzadar Fizzadar self-assigned this May 24, 2020
@Fizzadar
Copy link
Member

Fizzadar commented Jun 7, 2020

Finally managed to replicate this using just pywebview! I've added this issue: r0x0r/pywebview#543

Investigation continues into why.

@github-actions
Copy link

github-actions bot commented Jul 8, 2020

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Jul 8, 2020
@Fizzadar Fizzadar removed the Stale label Jul 8, 2020
@github-actions
Copy link

github-actions bot commented Aug 8, 2020

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Aug 8, 2020
@Fizzadar Fizzadar removed the Stale label Aug 9, 2020
@github-actions
Copy link

github-actions bot commented Sep 9, 2020

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Sep 9, 2020
@Fizzadar Fizzadar removed the Stale label Sep 9, 2020
@Fizzadar
Copy link
Member

Fizzadar commented Oct 7, 2020

Replaced by #140.

@Fizzadar Fizzadar closed this as completed Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Platform: Linux
Projects
None yet
Development

No branches or pull requests

2 participants