[ 
https://issues.apache.org/jira/browse/OFBIZ-3160?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jacques Le Roux closed OFBIZ-3160.
----------------------------------

       Resolution: Fixed
    Fix Version/s: SVN trunk
         Assignee: Jacques Le Roux

Thanks Jeremy,

You fix is implemented in trunk at r835161, R9.04 at r835162

> Possible NPE in security check
> ------------------------------
>
>                 Key: OFBIZ-3160
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3160
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework, order
>    Affects Versions: Release Branch 9.04
>            Reporter: Wickersheimer Jeremy
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: Release Branch 9.04, SVN trunk
>
>
> I ran into a NPE in one of our groovy script, it is due to a small issue in 
> OFBizSecurity, hasEntityPermission(String entity, String action, HttpSession 
> session) and hasEntityPermission(String entity, String action, GenericValue 
> userLogin) have the same signature in that case resulting in the first method 
> being called rom the groovy script.
> A very simple fix is to add a check on the session object: 
>     public boolean hasEntityPermission(String entity, String action, 
> HttpSession session) {
>         if (session == null) return false;
> (edit: remove second part, actually that is something we modified)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to