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

bspc control --subscribe will crash when unplugging external monitor #162

Closed
ghost opened this issue Jul 29, 2014 · 6 comments
Closed

bspc control --subscribe will crash when unplugging external monitor #162

ghost opened this issue Jul 29, 2014 · 6 comments

Comments

@ghost
Copy link

ghost commented Jul 29, 2014

For me bspc control --subscribe will crash, if I disconnect my external monitor from my laptop with the option 'remove_unplugged_monitors' set to 'true'.
It has to be something within the last 10 commits! Before those it is working fine here.

Don't really know, what log files to append, because it just stops without throwing an error message. Exit code is 0.

EDIT: Forgot to mention I use xrandr --output DP1 --off to disable the monitor after disconnecting

@baskerville
Copy link
Owner

gdb backtrace?

@ghost
Copy link
Author

ghost commented Jul 29, 2014

Tried to get the stack of bspc control --subscribe with gdb, but as soon as I disconnect the monitor it will crash and I can't really log the stack. Maybe you got a hint how to log this?

dmesg tells me this:
bspwm[28867]: segfault at 20 ip 0000000000410854 sp 00007fffc054cec0 error 4 in bspwm[400000+15000]

The last message I get is (which should be ok):
WMeDP1:FDesktop1:LT:mDP1:FDesktop2:LT

@baskerville
Copy link
Owner

#62 (comment)

@ghost
Copy link
Author

ghost commented Jul 29, 2014

I added if (m != NULL && m->desk != NULL) in line 90 of subscribe.c before the fprintf and the problem seems to be fixed!
At least it isn't crashing for me anymore

Tried to get a system-coredump, but failed.
Seems like it is not creating one when crashing.

@baskerville
Copy link
Owner

It should be fixed by 1e0a7c8.

@ghost
Copy link
Author

ghost commented Jul 29, 2014

Seems to be working.
Thanks!

@ghost ghost closed this as completed Jul 29, 2014
This issue was closed.
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

No branches or pull requests

1 participant