[
https://issues.apache.org/jira/browse/NET-544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sebb updated NET-544:
---------------------
Fix Version/s: 3.4
> initiateListParsing does not correctly check if parserKey was cached
> --------------------------------------------------------------------
>
> Key: NET-544
> URL: https://issues.apache.org/jira/browse/NET-544
> Project: Commons Net
> Issue Type: Bug
> Components: FTP
> Affects Versions: 3.3
> Reporter: Olivier Queyrut
> Priority: Minor
> Fix For: 3.4
>
>
> At line 3232 of FTPClient, it is mentionned that the parserKey and
> entryParser are cached "to avoid creation of a new object every time a file
> listing is generated".
> However the test seems to be incorrect as a new entryParser is created every
> time the method "listFiles" is called. Indeed, in method named listFiles, the
> initiateListParsing is called with a null argument for the parserKey.
> So the test : "if(__entryParser == null || !
> __entryParserKey.equals(parserKey))" is always true (even if __entryParserKey
> has been cached) and thus a new entryParser is created.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
