[Slim-Checkins] r9359 - in /branches/6.5/server/Slim/Player: Source.pm Squeezebox2.pm

[andy at svn.slimdevices.com]

Author: andy
Date: Fri Sep  1 18:07:12 2006
New Revision: 9359

URL: http://svn.slimdevices.com?rev=9359&view=rev
Log:
Bug 4038, Don't crash if there is no client for chunks.  Use stop method instead of execute when upgading firmware

Modified:
    branches/6.5/server/Slim/Player/Source.pm
    branches/6.5/server/Slim/Player/Squeezebox2.pm

Modified: branches/6.5/server/Slim/Player/Source.pm
URL: http://svn.slimdevices.com/branches/6.5/server/Slim/Player/Source.pm?rev=9359&r1=9358&r2=9359&view=diff
==============================================================================
--- branches/6.5/server/Slim/Player/Source.pm (original)
+++ branches/6.5/server/Slim/Player/Source.pm Fri Sep  1 18:07:12 2006
@@ -645,6 +645,8 @@
 	my $chunk;
 	my $len;
 
+	return if !$client;
+
 	# if there's a chunk in the queue, then use it.
 	if (ref($client->chunks) eq 'ARRAY' && scalar(@{$client->chunks})) {
 

Modified: branches/6.5/server/Slim/Player/Squeezebox2.pm
URL: http://svn.slimdevices.com/branches/6.5/server/Slim/Player/Squeezebox2.pm?rev=9359&r1=9358&r2=9359&view=diff
==============================================================================
--- branches/6.5/server/Slim/Player/Squeezebox2.pm (original)
+++ branches/6.5/server/Slim/Player/Squeezebox2.pm Fri Sep  1 18:07:12 2006
@@ -201,7 +201,7 @@
 		return(0);
 	}
 	
-	$client->execute(["stop"]);
+	$client->stop();
 
 	$::d_firmware && msg("using new update mechanism: $filename\n");