The following document contains the results of FindBugs Report
FindBugs Version is 1.1.1
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 363 | 27 | 0 | 2 |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.wiztools.jenkryptor.Globals.mode isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available |
| org.wiztools.jenkryptor.Globals.msgDisplayer isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available |
| org.wiztools.jenkryptor.Globals.files is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | Not available |
| org.wiztools.jenkryptor.Globals.MAIN_FRAME should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
| org.wiztools.jenkryptor.Globals.isRunning should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 40 |
| org.wiztools.jenkryptor.Globals.password should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Class org.wiztools.jenkryptor.MainJFrame defines non-transient non-serializable instance field FREEZE_KEY_LISTENER | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class org.wiztools.jenkryptor.MainJFrame defines non-transient non-serializable instance field FREEZE_MOUSE_LISTENER | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class org.wiztools.jenkryptor.MainJFrame defines non-transient non-serializable instance field validator | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Write to static field org.wiztools.jenkryptor.Globals.MAIN_FRAME from instance method org.wiztools.jenkryptor.MainJFrame.MainJFrame() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 115 |
| Write to static field org.wiztools.jenkryptor.Globals.msgDisplayer from instance method org.wiztools.jenkryptor.MainJFrame.MainJFrame() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 113 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method org.wiztools.jenkryptor.MainJFrame$2.accept(java.io.File) invokes dubious String.toUpperCase() or String.toLowerCase; use the Locale parameterized version instead | I18N | DM_CONVERT_CASE | 79 |
| The class org.wiztools.jenkryptor.MainJFrame$2 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Write to static field org.wiztools.jenkryptor.Globals.files from instance method org.wiztools.jenkryptor.MainJFrame$20.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 526 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class org.wiztools.jenkryptor.MainJFrame$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.wiztools.jenkryptor.MainJFrame$FreezeKeyListener be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.wiztools.jenkryptor.MainJFrame$FreezeMouseListener be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.wiztools.jenkryptor.Preferences.deleteSource_pref should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 23 |
| org.wiztools.jenkryptor.Preferences.overwriteDestination_pref should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 22 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Write to static field org.wiztools.jenkryptor.Preferences.deleteSource_pref from instance method org.wiztools.jenkryptor.PreferencesJPanel.jbOkActionPerformed(java.awt.event.ActionEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 123 |
| Write to static field org.wiztools.jenkryptor.Preferences.overwriteDestination_pref from instance method org.wiztools.jenkryptor.PreferencesJPanel.jbOkActionPerformed(java.awt.event.ActionEvent) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 122 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Write to static field org.wiztools.jenkryptor.Globals.isRunning from instance method org.wiztools.jenkryptor.Processor$1.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 53 |
| Write to static field org.wiztools.jenkryptor.Globals.isRunning from instance method org.wiztools.jenkryptor.Processor$1.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 77 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should org.wiztools.jenkryptor.Processor$ProcessThread be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.wiztools.jenkryptor.ProgressBarProvisioningManager.getLPEArr() may expose internal representation by returning org.wiztools.jenkryptor.ProgressBarProvisioningManager.lpe_arr | MALICIOUS_CODE | EI_EXPOSE_REP | 49 |
| Unread field: org.wiztools.jenkryptor.ProgressBarProvisioningManager.number | PERFORMANCE | URF_UNREAD_FIELD | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| org.wiztools.jenkryptor.util.FileUtil.getFileNamesCSV(java.io.File[]) may expose internal static state by storing a mutable object into a static field org.wiztools.jenkryptor.Globals.files | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 54 |