Olivier Queyrut created NET-544:
-----------------------------------

             Summary: 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


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.2#6252)

Reply via email to