Bug 1194 – Mozart Forever, Always station from Live365 crashes the server

Bug 1194 - Mozart Forever, Always station from Live365 crashes the server

Summary:

Mozart Forever, Always station from Live365 crashes the server

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Streaming To SlimServer
Version:	6.0.0
Platform:	All All

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-03-25 11:15 UTC by Kevin Pearsall
Modified:	2008-08-18 10:53 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
currentSong is already a single item (458 bytes, patch) 2005-03-25 12:48 UTC, KDF	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 Kevin Pearsall 2005-03-25 11:15:53 UTC

You get an error that says:
Can't use string ("live365://www.live365.com/play/m") as an ARRAY ref while
"strict refs" in use at /Users/kpearsall/6/server/Plugins/Live365.pm line 651.

STR:
- go to internet radio -> live365 -> search
- search for stations, enter MOZART
- press play on Mozart forever, always...  see crash

Comment 1 Dan Sully 2005-03-25 11:47:10 UTC

Are you running the latest nightly?

I'm piping out Mozart right now, no crash.

Comment 2 Kevin Pearsall 2005-03-25 11:48:20 UTC

running r2722

Comment 3 KDF 2005-03-25 12:20:36 UTC

i get the same crash

Comment 4 Kevin Pearsall 2005-03-25 12:22:05 UTC

oh.  this is customer reported also, btw...

Comment 5 KDF 2005-03-25 12:48:03 UTC

Created attachment 377 [details]
currentSong is already a single item

we're setting $currentSong to playList($client)->[0] earlier in teh code, so
the ->[0] isn't needed again at 651.  this stops the crash for me.

Comment 6 Kevin Pearsall 2005-03-25 13:38:55 UTC

patch works well for me.  (wow this is a very low bitrate station)

Comment 7 Dan Sully 2005-03-25 14:03:56 UTC

Fixed in subversion change 2727.