The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Exp
Effort is Max
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 83 | 36 | 0 | 0 |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.datamodel.Conference.getEndDate() may expose internal representation by returning Conference._endDate | MALICIOUS_CODE | EI_EXPOSE_REP | 118 |
| edu.drexel.cmsDb.datamodel.Conference.getStartDate() may expose internal representation by returning Conference._startDate | MALICIOUS_CODE | EI_EXPOSE_REP | 110 |
| edu.drexel.cmsDb.datamodel.Conference.Conference(String, String, String, String, Date, Date, String) may expose internal representation by storing an externally mutable object into Conference._endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 64 |
| edu.drexel.cmsDb.datamodel.Conference.Conference(String, String, String, String, Date, Date, String) may expose internal representation by storing an externally mutable object into Conference._startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 63 |
| edu.drexel.cmsDb.datamodel.Conference.setEndDate(Date) may expose internal representation by storing an externally mutable object into Conference._endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 121 |
| edu.drexel.cmsDb.datamodel.Conference.setStartDate(Date) may expose internal representation by storing an externally mutable object into Conference._startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 113 |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.datamodel.Paper.getlastModifiedDate() may expose internal representation by returning Paper._lastModifiedDate | MALICIOUS_CODE | EI_EXPOSE_REP | 154 |
| edu.drexel.cmsDb.datamodel.Paper.setlastModifiedDate(Date) may expose internal representation by storing an externally mutable object into Paper._lastModifiedDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 157 |
| Class edu.drexel.cmsDb.datamodel.Paper defines non-transient non-serializable instance field _originalPaper | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.datamodel.Review.getSubmittedDate() may expose internal representation by returning Review._submittedDate | MALICIOUS_CODE | EI_EXPOSE_REP | 91 |
| edu.drexel.cmsDb.datamodel.Review.setSubmittedDate(Date) may expose internal representation by storing an externally mutable object into Review._submittedDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 94 |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.datamodel.User.getCreateDate() may expose internal representation by returning User._createDate | MALICIOUS_CODE | EI_EXPOSE_REP | 96 |
| edu.drexel.cmsDb.datamodel.User.setCreateDate(Date) may expose internal representation by storing an externally mutable object into User._createDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 99 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Nullcheck of paper at line 195 of value previously dereferenced in edu.drexel.cmsDb.ejb.CmsDbBean.addReview(User, Review) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 194 |
| Class edu.drexel.cmsDb.ejb.CmsDbBean defines non-transient non-serializable instance field _em | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 109 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 166 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 232 |
| Method edu.drexel.cmsDb.ejb.CmsDbQueryBean.searchConference(User, SearchConferenceCriteriaVo) uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 131 |
| Method edu.drexel.cmsDb.ejb.CmsDbQueryBean.searchPaper(User, SearchPaperCriteriaVo) uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 193 |
| Method edu.drexel.cmsDb.ejb.CmsDbQueryBean.searhReview(User, SearchReviewCriteriaVo) uses Collection.toArray() with zero-length array argument | PERFORMANCE | ITA_INEFFICIENT_TO_ARRAY | 237 |
| Class edu.drexel.cmsDb.ejb.CmsDbQueryBean defines non-transient non-serializable instance field _em | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.getEndDate() may expose internal representation by returning SearchConferenceCriteriaVo._endDate | MALICIOUS_CODE | EI_EXPOSE_REP | 35 |
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.getStartDate() may expose internal representation by returning SearchConferenceCriteriaVo._startDate | MALICIOUS_CODE | EI_EXPOSE_REP | 59 |
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.SearchConferenceCriteriaVo(String, String, String, String, Date, Date) may expose internal representation by storing an externally mutable object into SearchConferenceCriteriaVo._endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 23 |
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.SearchConferenceCriteriaVo(String, String, String, String, Date, Date) may expose internal representation by storing an externally mutable object into SearchConferenceCriteriaVo._startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 22 |
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.setEndDate(Date) may expose internal representation by storing an externally mutable object into SearchConferenceCriteriaVo._endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 39 |
| edu.drexel.cmsDb.ejb.SearchConferenceCriteriaVo.setStartDate(Date) may expose internal representation by storing an externally mutable object into SearchConferenceCriteriaVo._startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 63 |
| Bug | Category | Details | Line |
|---|---|---|---|
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.getDateRangeEnd() may expose internal representation by returning SearchPaperCriteriaVo._dateRangeEnd | MALICIOUS_CODE | EI_EXPOSE_REP | 43 |
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.getDateRangeStart() may expose internal representation by returning SearchPaperCriteriaVo._dateRangeStart | MALICIOUS_CODE | EI_EXPOSE_REP | 51 |
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.SearchPaperCriteriaVo(String, String, String, String, Date, Date) may expose internal representation by storing an externally mutable object into SearchPaperCriteriaVo._dateRangeEnd | MALICIOUS_CODE | EI_EXPOSE_REP2 | 23 |
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.SearchPaperCriteriaVo(String, String, String, String, Date, Date) may expose internal representation by storing an externally mutable object into SearchPaperCriteriaVo._dateRangeStart | MALICIOUS_CODE | EI_EXPOSE_REP2 | 22 |
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.setDateRangeEnd(Date) may expose internal representation by storing an externally mutable object into SearchPaperCriteriaVo._dateRangeEnd | MALICIOUS_CODE | EI_EXPOSE_REP2 | 47 |
| edu.drexel.cmsDb.ejb.SearchPaperCriteriaVo.setDateRangeStart(Date) may expose internal representation by storing an externally mutable object into SearchPaperCriteriaVo._dateRangeStart | MALICIOUS_CODE | EI_EXPOSE_REP2 | 55 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Exception is caught when Exception is not thrown in edu.drexel.cmsDb.jmx.CmsDbMBean.addAdmin(String, String, String, String, String, String, String, String, String) | STYLE | REC_CATCH_EXCEPTION | 62 |
| Exception is caught when Exception is not thrown in edu.drexel.cmsDb.jmx.CmsDbMBean.addUser(String, String) | STYLE | REC_CATCH_EXCEPTION | 43 |
