commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Biestro (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (JEXL-239) Add NPE check to property for MapGetExecutor
Date Wed, 06 Sep 2017 16:01:00 GMT

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

Henri Biestro commented on JEXL-239:
------------------------------------

Backporting JEXL3 checks / fix

src/main/java/org/apache/commons/jexl2/internal/BooleanGetExecutor.java
src/main/java/org/apache/commons/jexl2/internal/DuckGetExecutor.java
src/main/java/org/apache/commons/jexl2/internal/DuckSetExecutor.java
src/main/java/org/apache/commons/jexl2/internal/MapGetExecutor.java
src/main/java/org/apache/commons/jexl2/internal/MapSetExecutor.java

Committed revision 1807495.

> Add NPE check to property for MapGetExecutor
> --------------------------------------------
>
>                 Key: JEXL-239
>                 URL: https://issues.apache.org/jira/browse/JEXL-239
>             Project: Commons JEXL
>          Issue Type: Improvement
>    Affects Versions: 2.1.1
>            Reporter: Bruno P. Kinoshita
>            Priority: Minor
>
> From GitHub pull request #2 https://github.com/apache/commons-jexl/pull/2
> {quote}
> When the MapGetExecutor is init by the key null, the property of MapGetExecutor will
be null.
> If the MapGetExecutor is cached and the key is changed (like map[index]), there will
throw an NPE.
> I think the intention for the condition is to compare the class compatibility, so I add
the NPE check here.
> {quote}



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

Mime
View raw message