[jira] [Commented] (OFBIZ-9973) [FB] Find Security Bugs

["Jacques Le Roux (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OFBIZ-9973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16244719#comment-16244719 ] 

Jacques Le Roux commented on OFBIZ-9973:
----------------------------------------

At revision: 1814642  in trunk and r1814646 in R16.11 I fixed 2 cases FB reported. 

They both relate to a request parameter that could be corrupted. They are respectively fixed using URLEncoder.encode() and File.getCanonicalFile()

Remains not fixed issues related with possible SQL injections that I'll possibly look at later...

> [FB] Find Security Bugs
> -----------------------
>
>                 Key: OFBIZ-9973
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9973
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: marketing, product
>    Affects Versions: Trunk, Release Branch 16.11
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>
> I recently [found|https://www.ysofters.com/2015/08/31/taint-analysis-added-to-findbugs/] FindBugs embeds an option [to Find Security Bugs|https://find-sec-bugs.github.io/]:
> I have tried this option: https://github.com/find-sec-bugs/find-sec-bugs/wiki/Eclipse-Tutorial
> Also later we should remember of OFBIZ-7963 and if possible run this tool in [Builbot using Gradle|https://search.maven.org/#search|gav|1|g:%22com.h3xstream.findsecbugs%22%20AND%20a:%22findsecbugs-plugin%22] (did not check feasibility)



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)