[ https://issues.apache.org/jira/browse/WW-4647?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Raintung Li updated WW-4647:
----------------------------
Description:
OGNL example: S2-029 leak:
#_memberAccess.excludedClasses=#{}.keySet() it has been fixed.
But now we can direct change the _memberAccess in the OGNLContext
#_memberAccess=@ognl.OgnlContext@DEFAULT_MEMBER_ACCESS
woo.. it can round the SecurityMemberAccess.isAccessible checking, because modify the OGNLContext
member _memberaccess that NOT check the accessible.
Struts should be self extend the OGNLContent to make securityOGNLContect to keep safe.
was:
OGNL example:
S2-029 leak:
#_memberAccess.excludedClasses=#{}.keySet()
But can direct change the _memberAccess in the OGNLContext
#_memberAccess=@ognl.OgnlContext@DEFAULT_MEMBER_ACCESS
woo.. it can round the SecurityMemberAccess.isAccessible checking, because it change the OGNLContext
member that NOT check the accessible.
Struts should be self extend the OGNLContent to make OGNLContect safe.
> Security: OGNL can change the MemberAccess in OGNLContext
> ---------------------------------------------------------
>
> Key: WW-4647
> URL: https://issues.apache.org/jira/browse/WW-4647
> Project: Struts 2
> Issue Type: Bug
> Components: Core Actions
> Affects Versions: 2.3.20
> Reporter: Raintung Li
> Priority: Critical
>
> OGNL example: S2-029 leak:
> #_memberAccess.excludedClasses=#{}.keySet() it has been fixed.
> But now we can direct change the _memberAccess in the OGNLContext
> #_memberAccess=@ognl.OgnlContext@DEFAULT_MEMBER_ACCESS
> woo.. it can round the SecurityMemberAccess.isAccessible checking, because modify the
OGNLContext member _memberaccess that NOT check the accessible.
> Struts should be self extend the OGNLContent to make securityOGNLContect to keep safe.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
|