Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 36340
Status: resolved
Priority: 0/
Queue: CGI

People
Owner: Nobody in particular
Requestors: mkelly [...] facet.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: Uninitialized value error in read_multipart
After upgrading from 3.29 to 3.37, I got errors when uploading a file: Use of uninitialized value $filename in substitution (s///) at (eval 2) line 23. I added the following "if statement" around line 3390: if (defined $filename) { $filename =~ s/^"([^"]*)"$/$1/; } This has fixed my immediate problem, although quite possibly not in the cleanest way. It looks like multiple form variables on the page were appearing the method. For example, the "submit" button was one of those appearing there, and didn't have a content disposition with "filename" in the string. This was a file upload with Internet Explorer 7.0.5730.11. The server is Fedora Core 9, which is using 3.37 as it's most current version.
On Sun Jun 01 18:10:45 2008, mkelly wrote: Show quoted text
> After upgrading from 3.29 to 3.37, I got errors when uploading a file: > > Use of uninitialized value $filename in substitution (s///) at (eval 2) > line 23. > > I added the following "if statement" around line 3390: > > if (defined $filename) { > $filename =~ s/^"([^"]*)"$/$1/; > } > > This has fixed my immediate problem, although quite possibly not in the > cleanest way. It looks like multiple form variables on the page were > appearing the method. For example, the "submit" button was one of those > appearing there, and didn't have a content disposition with "filename" > in the string. > > This was a file upload with Internet Explorer 7.0.5730.11. The server is > Fedora Core 9, which is using 3.37 as it's most current version.
Thanks for the report. Could you retest with 3.43 to see if the issue persists? Maybe this patch that was released since then may help: " 3. Added patch from Renee Baecker that fixes the inappropriate escaping of fields in multipart headers" Mark
No further feedback in multiple years. Considering resolved.