Rapport FindBugs de détection des bugs
Le document suivant contient les résultats du FindBugs
La version de FindBugs est 3.0.1
La limite est
L'effort est max
Résumé
| Classes |
Bugs |
Erreurs |
Classes Manquantes |
| 168 |
48 |
0 |
0 |
fr.paris.lutece.plugins.deployment.business.CommandResult
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| fr.paris.lutece.plugins.deployment.business.CommandResult.ERROR_TYPE_STOP n'est pas final mais devrait l'être |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
56 |
High |
| fr.paris.lutece.plugins.deployment.business.CommandResult.STATUS_ERROR n'est pas final mais devrait l'être |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
53 |
High |
| fr.paris.lutece.plugins.deployment.business.CommandResult.STATUS_OK n'est pas final mais devrait l'être |
MALICIOUS_CODE |
MS_SHOULD_BE_FINAL |
54 |
High |
fr.paris.lutece.plugins.deployment.business.DefaultAction
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| fr.paris.lutece.plugins.deployment.business.DefaultAction.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
80 |
Medium |
fr.paris.lutece.plugins.deployment.business.DumpAction
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| fr.paris.lutece.plugins.deployment.business.DumpAction.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
105 |
Medium |
fr.paris.lutece.plugins.deployment.business.InitAppContextAction
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.business.InitAppContextAction.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) |
STYLE |
DLS_DEAD_LOCAL_STORE |
75 |
Medium |
| fr.paris.lutece.plugins.deployment.business.InitAppContextAction.canRunAction(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
103 |
Medium |
| Found reliance on default encoding in fr.paris.lutece.plugins.deployment.business.InitAppContextAction.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]): String.getBytes() |
I18N |
DM_DEFAULT_ENCODING |
81 |
High |
fr.paris.lutece.plugins.deployment.business.InitMysqlDatabase
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.business.InitMysqlDatabase.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) |
STYLE |
DLS_DEAD_LOCAL_STORE |
68 |
Medium |
| fr.paris.lutece.plugins.deployment.business.InitMysqlDatabase.canRunAction(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
121 |
Medium |
| Found reliance on default encoding in fr.paris.lutece.plugins.deployment.business.InitMysqlDatabase.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]): String.getBytes() |
I18N |
DM_DEFAULT_ENCODING |
74 |
High |
fr.paris.lutece.plugins.deployment.business.SiteDeploy
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| La méthode fr.paris.lutece.plugins.deployment.business.SiteDeploy.getDeploymentDate() risque d'exposer sa représentation interne en renvoyant fr.paris.lutece.plugins.deployment.business.SiteDeploy._deploymentDate |
MALICIOUS_CODE |
EI_EXPOSE_REP |
80 |
Medium |
| La méthode fr.paris.lutece.plugins.deployment.business.SiteDeploy.setDeploymentDate(Date) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans fr.paris.lutece.plugins.deployment.business.SiteDeploy._deploymentDate |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
85 |
Medium |
fr.paris.lutece.plugins.deployment.business.SiteTag
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| La méthode fr.paris.lutece.plugins.deployment.business.SiteTag.getTagDate() risque d'exposer sa représentation interne en renvoyant fr.paris.lutece.plugins.deployment.business.SiteTag._tagDate |
MALICIOUS_CODE |
EI_EXPOSE_REP |
82 |
Medium |
| La méthode fr.paris.lutece.plugins.deployment.business.SiteTag.setTagDate(Date) risque d'exposer sa représentation interne en stockant un objet externe modifiable dans fr.paris.lutece.plugins.deployment.business.SiteTag._tagDate |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
77 |
Medium |
fr.paris.lutece.plugins.deployment.business.WarInstallerAction
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| fr.paris.lutece.plugins.deployment.business.WarInstallerAction.run(Application, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
113 |
Medium |
fr.paris.lutece.plugins.deployment.business.WorkflowDeploySiteContext
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| La classe fr.paris.lutece.plugins.deployment.business.WorkflowDeploySiteContext définit le champ d'instance fr.paris.lutece.plugins.deployment.business.WorkflowDeploySiteContext._vcsUser non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
High |
| La classe fr.paris.lutece.plugins.deployment.business.WorkflowDeploySiteContext définit le champ d'instance fr.paris.lutece.plugins.deployment.business.WorkflowDeploySiteContext._scriptFileItem non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
fr.paris.lutece.plugins.deployment.service.ActionService
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| fr.paris.lutece.plugins.deployment.service.ActionService.executeAction(Application, IAction, ServerApplicationInstance, CommandResult, ActionParameter[]) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
157 |
Medium |
fr.paris.lutece.plugins.deployment.service.ServerApplicationService
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Primitive value is boxed and then immediately unboxed in fr.paris.lutece.plugins.deployment.service.ServerApplicationService.getStatus(Application, ServerApplicationInstance) |
PERFORMANCE |
BX_BOXING_IMMEDIATELY_UNBOXED |
204 |
Medium |
| fr.paris.lutece.plugins.deployment.service.ServerApplicationService.getStatus(Application, ServerApplicationInstance) invoque le constructeur inutile Boolean() ; utilisez Boolean.valueOf(...) à la place |
PERFORMANCE |
DM_BOOLEAN_CTOR |
204 |
Medium |
| fr.paris.lutece.plugins.deployment.service.ServerApplicationService.getStatus(Application, ServerApplicationInstance) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead |
PERFORMANCE |
DM_NUMBER_CTOR |
204 |
Medium |
fr.paris.lutece.plugins.deployment.service.WorkflowDeploySiteService
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Test de nullité dans action d'une valeur préalablement déréférencée dans fr.paris.lutece.plugins.deployment.service.WorkflowDeploySiteService.executeServerAction(String, HttpServletRequest, WorkflowDeploySiteContext, Locale) |
CORRECTNESS |
RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE |
221 |
Medium |
fr.paris.lutece.plugins.deployment.service.vcs.SvnService
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.service.vcs.SvnService.doCheckoutSite(String, String, SvnUser, CommandResult, String, String) |
STYLE |
DLS_DEAD_LOCAL_STORE |
146 |
Medium |
| Test de nullité dans user d'une valeur préalablement déréférencée dans fr.paris.lutece.plugins.deployment.service.vcs.SvnService.doCheckoutSite(String, String, SvnUser, CommandResult, String, String) |
CORRECTNESS |
RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE |
125 |
Medium |
fr.paris.lutece.plugins.deployment.util.FTPUtils
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.util.FTPUtils.uploadFile(String, String, FtpInfo, String, CommandResult, boolean) |
STYLE |
DLS_DEAD_LOCAL_STORE |
211 |
Medium |
fr.paris.lutece.plugins.deployment.util.FileUtil
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Un pointeur à null renvoyé par une méthode risque d'être déréférencé dans fr.paris.lutece.plugins.deployment.util.FileUtil.delete(File, StringBuffer) |
STYLE |
NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE |
51 |
Medium |
| Un pointeur à null renvoyé par une méthode risque d'être déréférencé dans fr.paris.lutece.plugins.deployment.util.FileUtil.list(String, String, boolean, boolean) |
STYLE |
NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE |
87 |
Medium |
fr.paris.lutece.plugins.deployment.util.ReleaseSVNCheckoutClient
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Champ inutilisé : fr.paris.lutece.plugins.deployment.util.ReleaseSVNCheckoutClient.bCancelled |
PERFORMANCE |
URF_UNREAD_FIELD |
60 |
Medium |
fr.paris.lutece.plugins.deployment.util.URLUtils
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.util.URLUtils.checkUrl(String) |
STYLE |
DLS_DEAD_LOCAL_STORE |
108 |
High |
fr.paris.lutece.plugins.deployment.util.vcs.GitUtils
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Possible déréférencement d'un pointeur null dans le chemin d'exception de fr.paris.lutece.plugins.deployment.util.vcs.GitUtils.cloneOrReturnGit(String, String, CommandResult, GitUser, String, String) |
CORRECTNESS |
NP_NULL_ON_SOME_PATH_EXCEPTION |
119 |
Medium |
| La méthode fr.paris.lutece.plugins.deployment.util.vcs.GitUtils.checkAuthentication(String, String, String, GitUser) intercepte Exception, mais Exception n'est pas lancé dans le bloc try/catch et RuntimeException n'est pas appelé |
STYLE |
REC_CATCH_EXCEPTION |
443 |
Medium |
| La méthode fr.paris.lutece.plugins.deployment.util.vcs.GitUtils.getLastLog(Git, int) concatène des chaînes au moyen de + en boucle |
PERFORMANCE |
SBSC_USE_STRINGBUFFER_CONCATENATION |
254 |
Medium |
fr.paris.lutece.plugins.deployment.util.vcs.ReleaseSVNCommitClient
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Champ inutilisé : fr.paris.lutece.plugins.deployment.util.vcs.ReleaseSVNCommitClient.bCancelled |
PERFORMANCE |
URF_UNREAD_FIELD |
62 |
Medium |
fr.paris.lutece.plugins.deployment.util.vcs.ReleaseSVNCopyClient
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Champ inutilisé : fr.paris.lutece.plugins.deployment.util.vcs.ReleaseSVNCopyClient.bCancelled |
PERFORMANCE |
URF_UNREAD_FIELD |
62 |
Medium |
fr.paris.lutece.plugins.deployment.util.vcs.SVNUtils
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.util.vcs.SVNUtils.doSvnCheckoutSite(String, String, String, ReleaseSVNCheckoutClient, CommandResult) |
STYLE |
DLS_DEAD_LOCAL_STORE |
189 |
Medium |
fr.paris.lutece.plugins.deployment.web.DeploymentJspBean
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.doDeployApplication(HttpServletRequest) |
STYLE |
DLS_DEAD_LOCAL_STORE |
995 |
High |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.doDeployApplication(HttpServletRequest) |
STYLE |
DLS_DEAD_LOCAL_STORE |
983 |
Medium |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.getFormDeployApplication(HttpServletRequest) |
STYLE |
DLS_DEAD_LOCAL_STORE |
565 |
Medium |
| Alimentation à perte d'une variable locale dans la méthode fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.getFormDeployData(HttpServletRequest, WorkflowDeploySiteContext) |
STYLE |
DLS_DEAD_LOCAL_STORE |
1199 |
Medium |
| Le nom de la méthode fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.DoRemoveApplication(HttpServletRequest) ne commence pas par une minuscule |
BAD_PRACTICE |
NM_METHOD_NAMING_CONVENTION |
435-447 |
Medium |
| fr.paris.lutece.plugins.deployment.web.DeploymentJspBean.doDeployApplication(HttpServletRequest) may fail to clean up java.io.InputStream |
EXPERIMENTAL |
OBL_UNSATISFIED_OBLIGATION |
969 |
Medium |
| La classe fr.paris.lutece.plugins.deployment.web.DeploymentJspBean définit le champ d'instance fr.paris.lutece.plugins.deployment.web.DeploymentJspBean._filterDeployment non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
| La classe fr.paris.lutece.plugins.deployment.web.DeploymentJspBean définit le champ d'instance fr.paris.lutece.plugins.deployment.web.DeploymentJspBean._handler non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
fr.paris.lutece.plugins.deployment.web.DownloadServlet
| Bug |
Catégorie |
Détails |
Ligne |
Priorité |
| HTTP parameter directly written to HTTP header output in fr.paris.lutece.plugins.deployment.web.DownloadServlet.doGet(HttpServletRequest, HttpServletResponse) |
SECURITY |
HRS_REQUEST_PARAMETER_TO_HTTP_HEADER |
89 |
Medium |
| La classe fr.paris.lutece.plugins.deployment.web.DownloadServlet définit le champ d'instance fr.paris.lutece.plugins.deployment.web.DownloadServlet._applicationService non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
| La classe fr.paris.lutece.plugins.deployment.web.DownloadServlet définit le champ d'instance fr.paris.lutece.plugins.deployment.web.DownloadServlet._ftpService non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
| La classe fr.paris.lutece.plugins.deployment.web.DownloadServlet définit le champ d'instance fr.paris.lutece.plugins.deployment.web.DownloadServlet._serverApplicationService non transient et non sérialisable |
BAD_PRACTICE |
SE_BAD_FIELD |
Non disponible |
Medium |
