Attachment break when double quotes are used in pathname - tested on Firefox and Opera on Linux, you get this crash:
Use of uninitialized value in split at /home/twiki4/twikisvn/lib/TWiki/Sandbox.pm line 193.
at /usr/lib/perl5/5.8.5/CGI/Carp.pm line 314
CGI::Carp::realdie('Use of uninitialized value in split at /home/twiki4/twikisvn/...') called at /usr/lib/perl5/5.8.5/CGI/Carp.pm line 385
CGI::Carp::die('Use of uninitialized value in split at /home/twiki4/twikisvn/...') called at /home/twiki4/twikisvn/lib/TWiki/UI.pm line 85
TWiki::UI::__ANON__('Use of uninitialized value in split at /home/twiki4/twikisvn/...') called at /home/twiki4/twikisvn/lib/TWiki/Sandbox.pm line 193
TWiki::Sandbox::sanitizeAttachmentName('') called at /home/twiki4/twikisvn/lib/TWiki/UI/Upload.pm line 191
TWiki::UI::Upload::upload('TWiki=HASH(0x87ec7d8)') called at /home/twiki4/twikisvn/lib/TWiki/UI.pm line 160
TWiki::UI::__ANON__() called at /home/twiki4/twikisvn/lib/CPAN/lib//Error.pm line 379
eval {...} called at /home/twiki4/twikisvn/lib/CPAN/lib//Error.pm line 371
Error::subs::try('CODE(0x87d5d40)', 'HASH(0x91cdf24)') called at /home/twiki4/twikisvn/lib/TWiki/UI.pm line 198
TWiki::UI::run('CODE(0x8eafbbc)') called
--
TWiki:Main/RichardDonkin - 19 Jun 2007
- Works ok for me.....:
OK, here's a minimal test case:
-
echo hi >/tmp/hi.txt
- Attach this file, taking care to use double quotes in filename (inserted by Firefox on Linux for some filenames):
"/tmp/hi.txt"
- Get the crash
UPDATE: Now it's started working for some reason - seems to be intermittent, but I could definitely provoke it by adding double quotes just now... Does this site run with
SpeedyCGI or
ModPerl perhaps?
--
TWiki:Main.RichardDonkin - 19 Jun 2007
nope.
--
TWiki:Main.SvenDowideit - 19 Jun 2007
I think Richard was using the ~develop URL (which I just re-redirected to here), and this is a NOP. Please reopen it if you are still having problems, Richard.
CC
Thanks for redirecting - now, if you could have a look at
Item3652 and comment there, I could close off the
I18N updates for 4.2, which is what I was originally trying to do.
--
TWiki:Main.RichardDonkin - 21 Jun 2007