Bug 4212 - SVN revision 10016 gives root warning even if running as non-root user
: SVN revision 10016 gives root warning even if running as non-root user
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Misc
: 7.0
: PC Fedora
: P2 critical (vote)
: ---
Assigned To: Chris Owens
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-09-24 10:12 UTC by Lars Lind Nilsson
Modified: 2009-01-29 09:47 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Lind Nilsson 2006-09-24 10:12:56 UTC
I just SVN upgraded to 10017, and after the upgrade the Slimserver Service would not start. I'm trying to start the service with the service command (running as root). 

In the log I see this line: * Error: SlimServer must not be run as root! Exiting! *

Tried starting slimserver.pl directly as the slimserver user (using runuser), and I get the same error.

Downgraded to 10015, and the server starts without problems. Upgraded to 10016, and error appeared again.

I seems that revision 10016 broke something.
Comment 1 Michael Herger 2006-09-24 11:02:02 UTC
I think this is the scanner crashing. I'm currently experiencing the same issue while testing SlimCD: /tmp/slimserver.log is owned by root, with 644 permissions. I can't scan the collection from the web interface, but am getting "can't write to slimserver.log" in the log. The scanner does run from the shell, though, when printing logs to the console. 

If I chmod 666 the logfile, the scanner runs fine from the web interface as well.
Comment 2 Lars Lind Nilsson 2006-09-24 11:47:59 UTC
chmod of the slimserver.log didn't change anything. My slimserver.log is owned by the slimserver user.

But I just saw that a new change was checked in for the root issue, so I upgraded to 10021, and now my server can start again. Both when started as a service and when started directly. And when starting from root it seems that the user-context are changed to slimserver. 

So it seems that change 10020 fixed the problem.
Comment 3 Michael Herger 2006-09-24 12:16:28 UTC
That patch has not made it into the 6.5 branch (yet?). But copying over slimserver.pl and Slim/bootstrap.pl from the trunk seems to fix it.
Comment 4 KDF 2006-09-24 13:53:10 UTC
in trunk at change 10020
in 6.5 at change 10022