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
[archlinux] dde-session-daemon crashed when usb disk inserted #26
Comments
@snyh And also looks like a gir binding issue 😂 |
I just review and test the code, the problem is in dde-daemon not in the gir/gio-2.0. https://github.com/linuxdeepin/dde-daemon/blob/master/mounts/ifc.go#L113 the callback (which is a go pointer) will pass to cgo runtime, so in the callback we can't reference any go pointer. There have many code like this in dde code. It should be take carefully. The solution is not too complex, just don't reference any (even a simple pointer linke p:=new(int)) go pointer in the gobject callback. It can use the first parameter in callback and cast it to the correctly GObject Type, like this
https://github.com/linuxdeepin/dde-daemon/blob/master/mounts/ifc.go#L117 and this is also a bad smell. Don't unref the volume (or any way use the ref/unref like this, it should be use paired in one place, don't cross code block.), if you don't ref it. |
Any updates here? I am still getting lots of reports about this issue, as it's causing the whole DE including its core components (d-c-c and dock) stuck and almost unusable. |
The latest tag |
Well, looks not complete solved now 😂 |
Should be solved in dde-daemon 3.0.20. |
Unfortunately, this submission(#15502) will be reveted, because it was found to cause a serious bug: can not mount the iphone device. |
@Oberon2007 |
Thanks a lot @fasheng! Normally we just inherit Archlinux packages. Are you going to apply the patch, @felixonmars ? In case of deepin-daemon I have the problem that currently I'm not able to build it here as described due to this gopath error... |
Sure, I have included the patch in deepin-daemon 3.0.21-1 |
Unfortunately deepin-daemon 3.0.21 doesn't work at all here. Neither the dock nor launcher nor corner navigation are working as soon as the daemon is updated to 3.0.21! |
But both them works fine for me, I updated all the deepin packages to latest just now. How about you @felixonmars ? |
I'll try to collect some more detailed information. I also have all the latest arch packages installed. Downgrading deepin-daemon solves the issue for me and 3.0.20-2 works fine... |
With deepin-daemon 3.0.21 when trying to launch dde-dock I get tons of QPainter errors:
|
Any idea ;) @sbwtw |
@fasheng not sure, i guess its some dbus call blocked. by the way, it just warning, doesnt matter :P |
For some reason now it works - some other package update must have solved this...? |
This also casued the screen backlight be a kind of like a strobe.
@jouyouyun @hualet Did it reported before?
The text was updated successfully, but these errors were encountered: