You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Raintung Li (JIRA)" <ji...@apache.org> on 2016/06/17 03:52:05 UTC
[jira] [Updated] (WW-4647) Security: OGNL can change the
MemberAccess in OGNLContext
[ 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)