Bug 12491 - mkdir C:\WINDOWS\system32\C:\: Invalid argument
: mkdir C:\WINDOWS\system32\C:\: Invalid argument
Status: NEW
Product: Logitech Media Server
Classification: Unclassified
Component: Plugins
: 7.4.0
: PC Windows Home Server
: -- normal with 1 vote (vote)
: 8.0.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-22 14:11 UTC by Bonze
Modified: 2011-08-24 21:57 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 Bonze 2009-06-22 14:11:32 UTC
The filename, directory name, or volume label syntax is incorrect at /<C:\SqueezeCenter\server\SqueezeSvr.exe>Archive/Zip/DirectoryMember.pm line 62


Occurred after the recent change from 'SqueezeCenter' to 'Squeezebox Server'

7.4.27149 onwards

The error occurs if there is a plugin zip file in location Appdata\Squeezebox\Cache\DownloadedPlugins when the service is restarted.
The service will not restart until the folder is emptied.
Comment 1 Michael Herger 2009-06-23 00:29:43 UTC
Are you running SC as a service? If so: under your own user name or as the system user?
Comment 2 Bonze 2009-06-23 01:23:01 UTC
(In reply to comment #1)
> Are you running SC as a service? If so: under your own user name or as the
> system user?
Service is running as 'Local System account'
Comment 3 Michael Herger 2009-06-23 01:39:12 UTC
Ok, renaming was a red hering. It's a known bug with the latest version of Archive::Zip:

http://rt.cpan.org/Public/Bug/Display.html?id=47223

Alternatively we could disable the breaking code:

### Eclipse Workspace Patch 1.0
#P noweb-sqlite
Index: server/CPAN/Archive/Zip.pm
===================================================================
--- server/CPAN/Archive/Zip.pm	(revision 27203)
+++ server/CPAN/Archive/Zip.pm	(working copy)
@@ -538,8 +538,8 @@
 	$filename = '' unless defined($filename);
 	my $localDirs = @paths?File::Spec->catdir(@paths):'';
 	my $localName = File::Spec->catpath( $volume, $localDirs, $filename );
-    use Cwd;
-    $localName = File::Spec->catfile(getcwd, $localName) unless $volume;
+#    use Cwd;
+#    $localName = File::Spec->catfile(getcwd, $localName) unless $volume;
 	return $localName;
 }
Comment 4 Andy Grundman 2009-06-23 05:23:43 UTC
I'll see about fixing Archive::Zip.
Comment 5 Andy Grundman 2009-07-29 14:59:26 UTC
Moving 7.4 bugs to 8.0.
Comment 6 Michael Herger 2009-07-30 00:17:54 UTC
Andy - a new version 1.30 has been released about a month ago. It claims to fix/work around this issue. We might want to test the update.
Comment 7 Michael Herger 2009-07-30 04:35:30 UTC
Just noticed you have already updated to 1.29 which contains that fix (noweb-sqlite only). Can this bug be considered fixed?
Comment 8 Andy Grundman 2009-07-30 06:21:13 UTC
Probably, is this a serious enough issue to backport to trunk?
Comment 9 Michael Herger 2009-07-30 07:09:46 UTC
> Probably, is this a serious enough issue to backport to trunk?

Not sure: I think the version used in trunk is older than the breaking  
change anyway.
Comment 10 Michael Herger 2011-08-24 21:57:03 UTC
Bonze - is this still an issue?