The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
186 | 17 | 0 | 0 |
Bug | Category | Details | Line |
---|---|---|---|
Possible doublecheck on com.atlassian.mail.MailFactory.serverManager in com.atlassian.mail.MailFactory.getServerManager() | MT_CORRECTNESS | DC_DOUBLECHECK | 31-33 |
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 562 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 153 |
com.atlassian.mail.MailUtils.createZippedAttachmentMimeBodyPart(String) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 446 |
com.atlassian.mail.MailUtils.createZippedAttachmentMimeBodyPart(String) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 443 |
Bug | Category | Details | Line |
---|---|---|---|
com.atlassian.mail.MailUtils$Attachment.getContents() may expose internal representation by returning MailUtils$Attachment.contents | MALICIOUS_CODE | EI_EXPOSE_REP | 103 |
com.atlassian.mail.MailUtils$Attachment.Attachment(String, String, byte[]) may expose internal representation by storing an externally mutable object into MailUtils$Attachment.contents | MALICIOUS_CODE | EI_EXPOSE_REP2 | 93 |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.atlassian.mail.config.ConfigLoader.ConfigLoader(String) | STYLE | REC_CATCH_EXCEPTION | 58 |
Bug | Category | Details | Line |
---|---|---|---|
com.atlassian.mail.config.PropertiesLoader.ATLASSIAN_MAIL_PROPERTIES isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 23 |
Bug | Category | Details | Line |
---|---|---|---|
com.atlassian.mail.queue.AbstractMailQueueItem.getDateQueued() may expose internal representation by returning AbstractMailQueueItem.dateQueued | MALICIOUS_CODE | EI_EXPOSE_REP | 33 |
Bug | Category | Details | Line |
---|---|---|---|
Class com.atlassian.mail.queue.SingleMailQueueItem defines non-transient non-serializable instance field email | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.atlassian.mail.queue.SingleMailQueueItem is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
com.atlassian.mail.server.MailServerManager.SERVER_TYPES is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 25 |
Bug | Category | Details | Line |
---|---|---|---|
com.atlassian.mail.server.impl.PopMailServerImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Possible null pointer dereference of jndiSession in com.atlassian.mail.server.impl.SMTPMailServerImpl.getSession() | CORRECTNESS | NP_NULL_ON_SOME_PATH | 117 |
com.atlassian.mail.server.impl.SMTPMailServerImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | Not available |
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.atlassian.mail.server.managers.XMLMailServerManager.configure() | STYLE | REC_CATCH_EXCEPTION | 69 |