[ https://issues.apache.org/jira/browse/JAMES-3709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17481693#comment-17481693 ]
Benoit Tellier commented on JAMES-3709: --------------------------------------- > Admittedly this is a rare case, I only observed this during heavy load > testing. Do you still have that stacktrace around? I would be interested to have a closer look at it... Even upon load testing, this is kind of... Unexpected! > Missing null check in POP3 TopCmdHandler > ---------------------------------------- > > Key: JAMES-3709 > URL: https://issues.apache.org/jira/browse/JAMES-3709 > Project: James Server > Issue Type: Bug > Components: protocols > Affects Versions: master > Reporter: Karsten Otto > Priority: Major > Time Spent: 10m > Remaining Estimate: 0h > > TopCmdHandler is missing a null check when retrieving the message content, > i.e. the InputStream can be null, which results in an NPE down the line. > Admittedly this is a rare case, I only observed this during heavy load > testing. > On closer inspection it turns out that TopCmdHandler duplicates a major > portion of code from RetrCmdHandler, but with subtle differences such as the > null check mentioned above. > I propose to refactor the two classes to make them share the code, and > provide unit tests for all possible error/success cases. -- This message was sent by Atlassian Jira (v8.20.1#820001) --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org