uos / sick_tim

A ROS driver for the SICK TiM series of laser scanners.
http://wiki.ros.org/sick_tim
47 stars 90 forks source link

udev rule does not fully work for TIM551 #17

Closed ibaranov-cp closed 9 years ago

ibaranov-cp commented 10 years ago

Hey,

I could not get proper permissions on the TIM551 (kept having to do chmod 666 /usb/002/003)

until I modified your included udev rule to:

SUBSYSTEM=="usb", ACTION=="add", ATTR{idVendor}=="19a2", ATTR{idProduct}=="5001", GROUP="plugdev", MODE="666"

jspricke commented 10 years ago

I guess you are not in the plugdev group.

ibaranov-cp commented 10 years ago

Would it not be easier to add the mode 666 so that regardless it works for everyone? On 22 Sep 2014 16:52, "Jochen Sprickerhof" notifications@github.com wrote:

I guess you are not in the plugdev group.

— Reply to this email directly or view it on GitHub https://github.com/uos/sick_tim/issues/17#issuecomment-56441135.

jspricke commented 10 years ago

Feel free to do so, but I would not recommend it for everyone, as it's a security problem. Maybe we should add a hint to the documentation about the plugdev group. Feel free to send a patch.

ibaranov-cp commented 10 years ago

Forgive the ignorance on my part, but how is 666 on a USB port a security problem? (For my own knowledge :)

jspricke commented 10 years ago

It means everyone on the PC (might be remote) can access the port (i.e. write to it).