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

Fix for handling the G_IO_STATUS_ERROR #41

Closed
wants to merge 1 commit into from
Closed

Fix for handling the G_IO_STATUS_ERROR #41

wants to merge 1 commit into from

Conversation

z1gc
Copy link

@z1gc z1gc commented Jul 13, 2023

The Glib may returns G_IO_STATUS_ERROR without setting the GError** error, making the i3ipc looping forever. This may happens in restarting i3wm multiple times.

When the G_IO_STATUS_ERROR returned, continue the signal IPC shutdown process.

denesb added a commit to denesb/xfce4-i3-workspaces-plugin that referenced this pull request Jul 14, 2023
An old problem is that the plugin cannot work normally if the i3wm starts before xfce4-panel as in #62, here we're using a timer to handle the ipc resources' acquisition.

May still have some segfault there.

Tested in archlinux with latest xfce4-panel (v4.18.4).

Investigating an error that causing the .xsession-errors during restart i3wm twice or more, crashed in g_io_channel_read_chars: and channel->is_readable.

Backtrace seems indicates an error of i3ipc-glib.

The problem is caused by an unhandled error status code from glib, submitted a PR to altdesktop/i3ipc-glib#41, but may not get merged due to unmaintained.
@z1gc z1gc closed this by deleting the head repository Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant