[FTPSERVER-378] [FindBugs] Method DefaultFtpStatistics::setLogin() call passes null for nonnull parameter of new DefaultFtpStatistics$UserLogins(InetAddress)

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Fix Version/s: 1.0.0-M1, 1.1.5
• Component/s: Core
• Labels: findbugs npe
• Assignee: Unassigned
• Reporter: Sergey Vladimirov
• Created: 13/Jun/10 17:59
• Updated: 13/Mar/22 23:56

Description

If session.getRemoteAddress() is not instanceof InetSocketAddress, then address is null. But UserLogins do not accept nulls as argument (null can't be a key for ConcurrentHashMap)

I think 'fake' NULL address constant shall be introduced in DefaultFtpStatistics for this case.