You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by "Karsten Otto (Jira)" <se...@james.apache.org> on 2022/01/25 09:20:00 UTC
[jira] [Created] (JAMES-3709) Missing null check in POP3 TopCmdHandler
Karsten Otto created JAMES-3709:
-----------------------------------
Summary: 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
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-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org