Bug 1399 – slimserver should write the pid file before changing uid

Bug 1399 - slimserver should write the pid file before changing uid

Summary:

slimserver should write the pid file before changing uid

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Platform Support
Version:	unspecified
Platform:	Other All

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Dan Sully

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-04-19 21:11 UTC by Brooks Davis
Modified:	2008-08-18 10:54 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Fix for pid issue (1.47 KB, patch) 2005-06-24 08:43 UTC, Dan Sully	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Brooks Davis 2005-04-19 21:11:44 UTC

It would be useful if the slimserver wrote the pid file before setting it's  
uid.  That way, pid files could be put in /var/run where they belong while  
still setting the uid for security.  As it is I'm going to have to have the 
FreeBSD startup script create a sub directory to put it it.

Comment 1 Brooks Davis 2005-04-21 12:44:51 UTC

I just noticed a couple more relevent things here.  First, this was not 
the behavior in 5.x.  Second, there's a similar problem with log file.  It 
used to be possible to set the log file to /var/log/slimserver and have 
it just work.   Now you have to insure the log file exists and is owned 
by who ever is running the server if they can't write to the directory.

Comment 2 Blackketter Dean 2005-06-10 17:20:24 UTC

dan has, in his multitude of subversion check-outs, a fix for this

Comment 3 Dan Sully 2005-06-24 08:43:26 UTC

Created attachment 568 [details]
Fix for pid issue

Brooks - could you try this patch?

Thanks

Comment 4 Dan Sully 2005-07-07 16:18:51 UTC

Brooks - I've commited the change as subversion 3641. Please let me know if this works for you.

Comment 5 Brooks Davis 2005-07-22 16:06:27 UTC

6.1.1 works as expected.  Thanks for fixing this.  Sorry about the lack of 
response previously. 
 
-- Brooks

Comment 6 Chris Owens 2008-03-11 11:28:10 UTC

This bug was marked resolved in Slimserver 6.1, which is several versions ago.  If you're still seeing this bug, please re-open it.  Thanks!