Skip Menu |
 

This queue is for tickets about the CGI-Uploader CPAN distribution.

Report information
The Basics
Id: 66360
Status: open
Priority: 0/
Queue: CGI-Uploader

People
Owner: Nobody in particular
Requestors: MARKSTOS [...] cpan.org
Cc:
AdminCc:

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



Subject: upload() should close un-used $tmp_fh
Download (untitled) / with headers
text/plain 550b
Based on my experience working with a private fork of this project, I think the upload() method should call close() on the $tmp_fh. I found that when our fork was used in a long running process, the max limit of open files was eventually reached. This may not come up in the average use of web development, but appears to have an easy fix with no downsides. A test script could be devised which calls uploads multiple times, while a second terminal is used to call "lsof" or "fstat" to see if the open file count is increasing or constant.
Internal reference: RT#8770.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.