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

Chromium and libudev-compat #58

Open
fbt opened this issue Aug 30, 2015 · 8 comments
Open

Chromium and libudev-compat #58

fbt opened this issue Aug 30, 2015 · 8 comments

Comments

@fbt
Copy link
Contributor

fbt commented Aug 30, 2015

Chromium doesn't work with vdev's libudev. Here's a strace.

Not sure if it's chromium or libudev-compat being silly, but as it works with eudev, I've decided to poke you first.

@onimatrix81
Copy link

Yeah, google-chrome-44.0.2403.155_p1 dies for me too.
[13504:13504:0830/175408:ERROR:sandbox_linux.cc(345)] InitializeSandbox() called with multiple threads in process gpu-process

@jcnelson
Copy link
Owner

Thanks for the strace! The lines at the very end look interesting:

11004: [    libudev-fs.c:0416] udev_monitor_fs_setup: mkdir('/dev/metadata/udev/events/libudev-11004/') rc = -13
11004: [    libudev-fs.c:0570] udev_monitor_fs_destroy: open('/dev/metadata/udev/events/libudev-11004/') rc = -2
11004: [libudev-monitor.c:0130] udev_monitor_new_from_filesystem: udev_monitor_fs_setup() rc = -13

A bit of background: libudev-compat works by creating a PID-specific directory in /dev/metadata/udev/events/ and watching it for new files to be created. Each file will contain a uevent-like packet of KEY=VALUE pairs that can be parsed into a struct udev_device. The udev-compat.sh helper in vdev is responsible for querying vdev's /dev/metadata database and and generating these files.

It looks like what's happening here is that /dev/metadata/udev/events is not writeable to Chromium. Can you verify whether or not this is the case? dev-setup.sh is responsible for making this directory, and it looks like it's not creating it world-writeable (or writeable to a hypothetical group of hotplug-subscribers).

@onimatrix81
Copy link

chmod -r 777 /dev/metadata/udev/events removed the problem for me.

@onimatrix81
Copy link

Oh and yeah, there permissions were as this before the chmod:
drwxr-xr-x 6 root root 120 30. 8. 18:50 .
drwxr-xr-x 6 root root 140 30. 8. 07:52 ..
drwxr-xr-x 2 root root 40 30. 8. 20:15 global
drwx------ 2 root virgini 2.0K 30. 8. 20:15 libudev-7400
drwx------ 2 root virgini 2.0K 30. 8. 20:15 libudev-7401
drwx------ 2 root root 40 30. 8. 20:15 libudev-7629

@fbt
Copy link
Contributor Author

fbt commented Aug 30, 2015

Yep. I feel stupid for not checking now. /dev/metadata/udev/events is being crated with root:root 755.

@jcnelson
Copy link
Owner

I feel stupid for not explicitly chmod-ing the directories. This has been fixed as of 9aed0db.

@fbt
Copy link
Contributor Author

fbt commented Aug 31, 2015

It works with 777, but as with any tmp-like dir, it's better to make it 1777, so that other users can't remove your stuff.

@jcnelson
Copy link
Owner

@fbt Good catch; just pushed a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants