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

XIM: crash when restart fcitx5 #455

Closed
quininer opened this issue Apr 10, 2018 · 2 comments · Fixed by #451
Closed

XIM: crash when restart fcitx5 #455

quininer opened this issue Apr 10, 2018 · 2 comments · Fixed by #451
Labels
B - bug Dang, that shouldn't have happened DS - x11

Comments

@quininer
Copy link

quininer commented Apr 10, 2018

(lldb) run
Process 10240 launched: './target/debug/xim' (x86_64)
'中'
'文'
Process 10240 stopped
* thread #1, name = 'xim', stop reason = signal SIGSEGV: invalid address (fault address: 0xc)
    frame #0: 0x000000000000000c
error: memory read failed for 0x0
(lldb) bt
* thread #1, name = 'xim', stop reason = signal SIGSEGV: invalid address (fault address: 0xc)
  * frame #0: 0x000000000000000c
    frame #1: 0x000055555558528c xim`winit::platform::platform::x11::util::Ime::focus::hbfa261f0340a32c8(self=&0x7ffff6a4e730) at util.rs:433
    frame #2: 0x0000555555575416 xim`winit::platform::platform::x11::EventsLoop::process_event::h85eff9d31b0a72a9(self=&0x7fffffffac20, xev=&0x7fffffffaa88, callback=&0x7fffffffab50) at mod.rs:740
    frame #3: 0x000055555557130d xim`winit::platform::platform::x11::EventsLoop::run_forever::h6b17eaaf1b8e99f9(self=&0x7fffffffac20, callback=closure {

}) at mod.rs:198
    frame #4: 0x000055555556896c xim`winit::platform::platform::EventsLoop::run_forever::h0c8a88fe33696cf5(self=&0x7fffffffac18, callback=closure {

}) at mod.rs:415
    frame #5: 0x000055555556c1e5 xim`winit::EventsLoop::run_forever::h5f211e87314dd7ff(self=&0x7fffffffac18, callback=closure {

}) at lib.rs:231
    frame #6: 0x0000555555568aec xim`xim::main::he30972e73ceb8176 at main.rs:22
    frame #7: 0x000055555556c992 xim`std::rt::lang_start::_$u7b$$u7b$closure$u7d$$u7d$::hc9af16270bd23cea at rt.rs:74
    frame #8: 0x00005555556cf5e8 xim`std::panicking::try::do_call::h1fa5723294be8945 [inlined] std::rt::lang_start_internal::_$u7b$$u7b$closure$u7d$$u7d$::h0c81b3aba8d97d61 at rt.rs:59
    frame #9: 0x00005555556cf5dc xim`std::panicking::try::do_call::h1fa5723294be8945 at panicking.rs:306
    frame #10: 0x00005555556df22f xim`__rust_maybe_catch_panic at lib.rs:102
    frame #11: 0x00005555556c8496 xim`std::rt::lang_start_internal::hb77963e32cb566da [inlined] std::panicking::try::haa6d3d8e691373ac at panicking.rs:285
    frame #12: 0x00005555556c8460 xim`std::rt::lang_start_internal::hb77963e32cb566da [inlined] std::panic::catch_unwind::h48b33a2de2f7dd38 at panic.rs:361
    frame #13: 0x00005555556c8460 xim`std::rt::lang_start_internal::hb77963e32cb566da at rt.rs:58
    frame #14: 0x000055555556c972 xim`std::rt::lang_start::h863f61b00259401f(main=&0x555555568980, argc=1, argv=&0x7fffffffe298) at rt.rs:74
    frame #15: 0x0000555555568b7e xim`main + 46
    frame #16: 0x00007ffff7200f4a libc.so.6`__libc_start_main + 234
    frame #17: 0x000055555556424a xim`_start + 42
@francesca64 francesca64 added DS - x11 B - bug Dang, that shouldn't have happened labels Apr 10, 2018
@francesca64
Copy link
Member

Does this happen with #451?

@quininer
Copy link
Author

No, it works well! 🎉

francesca64 added a commit that referenced this issue Apr 11, 2018
Fixes #195
Fixes #277
Fixes #455

* Read `XMODIFIERS` explicitly/directly instead of calling `XSetLocaleModifiers` with an
empty string. This is useful for debugging purposes, and more clear to read and handle.
* Fallback to local input method if the one specified in `XMODIFIERS` is later closed on the
server end (i.e. if ibus/fcitx is terminated). Previously, that would cause the event loop
to freeze and usually also segfault.
* If using the fallback input method, respond to the `XMODIFIERS` input method later
becoming available. This means that the input method restarting is handled, and that even if
the program was started while ibus/fcitx/etc. was unavailable, it will start using it as
soon as it becomes available.
* Only one input method is opened for the whole event loop, with each window having its own
input context.
* IME works completely out of the box now, no longer requiring application developers to
call `setlocale` or `XSetLocaleModifiers`.
* Detailed error messages are provided if no input method could be opened. However, no
information is provided to the user if their intended `XMODIFIERS` input method failed to
open but the fallbacks (which will ostensibly always succeed) succeeded; in my opinion, this
is something that is best filled by adding a logging feature to winit.
tmfink pushed a commit to tmfink/winit that referenced this issue Jan 5, 2022
…ples/canvas_webgl_minimal/www/elliptic-6.5.4, r=jdm

Bump elliptic from 6.5.2 to 6.5.4 in /examples/canvas_webgl_minimal/www

Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.2 to 6.5.4.
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/indutny/elliptic/commit/43ac7f230069bd1575e1e4a58394a512303ba803"><code>43ac7f2</code></a> 6.5.4</li>
<li><a href="https://github.com/indutny/elliptic/commit/f4bc72be11b0a508fb790f445c43534307c9255b"><code>f4bc72b</code></a> package: bump deps</li>
<li><a href="https://github.com/indutny/elliptic/commit/441b7428b0e8f6636c42118ad2aaa186d3c34c3f"><code>441b742</code></a> ec: validate that a point before deriving keys</li>
<li><a href="https://github.com/indutny/elliptic/commit/e71b2d9359c5fe9437fbf46f1f05096de447de57"><code>e71b2d9</code></a> lib: relint using eslint</li>
<li><a href="https://github.com/indutny/elliptic/commit/8421a01aa3ff789c79f91eaf8845558a7be2b9fa"><code>8421a01</code></a> build(deps): bump elliptic from 6.4.1 to 6.5.3 (<a href="https://github-redirect.dependabot.com/indutny/elliptic/issues/231">#231</a>)</li>
<li><a href="https://github.com/indutny/elliptic/commit/8647803dc3d90506aa03021737f7b061ba959ae1"><code>8647803</code></a> 6.5.3</li>
<li><a href="https://github.com/indutny/elliptic/commit/856fe4d99fe7b6200556e6400b3bf585b1721bec"><code>856fe4d</code></a> signature: prevent malleability and overflows</li>
<li>See full diff in <a href="https://github.com/indutny/elliptic/compare/v6.5.2...v6.5.4">compare view</a></li>
</ul>
</details>
<br />

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=elliptic&package-manager=npm_and_yarn&previous-version=6.5.2&new-version=6.5.4)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
- `@dependabot use these labels` will set the current labels as the default for future PRs for this repo and language
- `@dependabot use these reviewers` will set the current reviewers as the default for future PRs for this repo and language
- `@dependabot use these assignees` will set the current assignees as the default for future PRs for this repo and language
- `@dependabot use this milestone` will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/servo/pathfinder/network/alerts).

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B - bug Dang, that shouldn't have happened DS - x11
Development

Successfully merging a pull request may close this issue.

2 participants