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 |
|---|---|---|---|
| 177 | 16 | 0 | 0 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Possible doublecheck on com.atlassian.mail.MailFactory.serverManager in com.atlassian.mail.MailFactory.getServerManager() | MT_CORRECTNESS | DC_DOUBLECHECK | 24-26 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 566 |
| com.atlassian.mail.MailUtils.createZippedAttachmentMimeBodyPart(String) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 400 |
| com.atlassian.mail.MailUtils.createZippedAttachmentMimeBodyPart(String) may fail to close stream on exception | BAD_PRACTICE | OS_OPEN_STREAM_EXCEPTION_PATH | 397 |
| Bug | Category | Details | Line |
|---|---|---|---|
| com.atlassian.mail.MailUtils$Attachment.getContents() may expose internal representation by returning MailUtils$Attachment.contents | MALICIOUS_CODE | EI_EXPOSE_REP | 97 |
| 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 | 87 |
| 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 | 18 |
| 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 |
