Skip Menu |

This queue is for tickets about the Net-FTPServer CPAN distribution.

Report information
The Basics
Id: 27640
Status: resolved
Priority: 0/
Queue: Net-FTPServer
People
Owner: ryochin [...] cpan.org
Requestors: asw [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.122
Fixed in: (no value)

History Show all quoted text
  Tue Jun 19 08:02:46 2007 asw [...] cpan.org - Ticket created
Subject: MLST is broken
MLST commands do not work. in FTPServer.pm, _MLST_command(), dir() is called on a filehandle object. $self->{cwd} is a dir handle. attached patch fixes the problem.
Subject: mlst_patch.txt
--- FTPServer.pm 2007-06-19 12:59:47.610741000 +0100 +++ FTPServer.pm.hack 2007-06-19 12:59:09.967037000 +0100 @@ -6741,8 +6741,8 @@ # If not file name is given, then we need to return # status on the current directory. Else we return # status on the file or directory name given. - my $fileh = $self->{cwd}; - my $dirh = $fileh->dir; + my $dirh = $self->{cwd}; + my $fileh; my $filename = "."; if ($rest ne "")
  Tue Jun 19 09:19:30 2007 RWMJ [...] cpan.org - Correspondence added
From: RWMJ [...] cpan.org
On Tue Jun 19 08:02:46 2007, ASW wrote: Show quoted text
> MLST commands do not work. > in FTPServer.pm, _MLST_command(), dir() is called on a filehandle > object. $self->{cwd} is a dir handle. > attached patch fixes the problem.
I have posted this to the Net::FTPServer mailing list for review and comment. Rich.
  Tue Jun 19 09:19:31 2007 The RT System itself - Status changed from 'new' to 'open'
  Tue Jun 19 10:35:53 2007 sobakasu [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #27640] MLST is broken
Date: Tue, 19 Jun 2007 15:35:36 +0100
To: bug-Net-FTPServer [...] rt.cpan.org
From: Andrew <sobakasu [...] gmail.com>
thanks. re-reading my bug report, it doesn't make much sense. The current code treats $self->{cwd} as a filehandle, but it is a dirhandle. hopefully it's clear from the patch what i'm talking about. On 6/19/07, via RT <bug-Net-FTPServer@rt.cpan.org> wrote: Show quoted text
> > <URL: http://rt.cpan.org/Ticket/Display.html?id=27640 > > > On Tue Jun 19 08:02:46 2007, ASW wrote:
> > MLST commands do not work. > > in FTPServer.pm, _MLST_command(), dir() is called on a filehandle > > object. $self->{cwd} is a dir handle. > > attached patch fixes the problem.
> > I have posted this to the Net::FTPServer mailing list for review and > comment. > > Rich. >
  Tue Nov 13 00:15:52 2012 ryochin [...] cpan.org - Correspondence added
applied & fixed in 1.123, thanks. https://github.com/ryochin/p5-net-ftpserver/commit/a717a2c2f793a8efb798feb06144856e05d291a6
  Tue Nov 13 00:15:53 2012 ryochin [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Nov 13 00:15:53 2012 ryochin [...] cpan.org - Given to RYOCHIN