Bug 537 – slimserver crash when d_cli set, using my client script

Bug 537 - slimserver crash when d_cli set, using my client script

Summary:

slimserver crash when d_cli set, using my client script

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	CLI
Version:	5.x or older
Platform:	PC All

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Blackketter Dean

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2004-09-07 05:07 UTC by Rob MacLaclan
Modified:	2008-12-18 11:51 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
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 Rob MacLaclan 2004-09-07 05:07:00 UTC

When debugging a simple CLI client script (posted on developer list),
I was able to reliably crash the server by setting d_cli and 
then sending a command.

Use of uninitialized value in subroutine entry at
/usr/local/slimserver/Slim/Control/CLI.pm line 158, <GEN53> line 1.
Bad arg length for Socket::inet_ntoa, length is 0, should be 4 at
/usr/local/slimserver/Slim/Control/CLI.pm line 158, <GEN53> line 1.

I think this is due to the fact that my script immediately closes
the socket after stuffing the command.  If I added a sleep call,
it didn't happen, and it also only happened if the url was not
actually playable, presumably because playing a valid url 
added some helpful delay in the server.

Comment 1 KDF 2004-09-21 12:37:47 UTC

fixed in cvs Sept 11, 2004

Comment 2 Chris Owens 2006-06-16 14:40:32 UTC

There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.

Comment 3 Chris Owens 2008-12-18 11:51:40 UTC

Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.