commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitri Blinov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (JEXL-221) Sporadic undefined property error caused by NPE at MapGetExecutor.tryInvoke()
Date Thu, 09 Mar 2017 07:20:38 GMT

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

Dmitri Blinov commented on JEXL-221:
------------------------------------

Do you think we could also cross-check other Get|SetExecutors for similar bugs?

> Sporadic undefined property error caused by NPE at MapGetExecutor.tryInvoke()
> -----------------------------------------------------------------------------
>
>                 Key: JEXL-221
>                 URL: https://issues.apache.org/jira/browse/JEXL-221
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 3.0
>            Reporter: Dmitri Blinov
>            Assignee: Henri Biestro
>
> I'm getting sporadic undefined property error when accessing hashmap by a key. The stack
trace tells the cause is NPE in MapGetExecutor.tryInvoke() at the following line:
> {code}
>         if (obj != null && method != null
>                 && objectClass.equals(obj.getClass())
>                 && (key == null || property.getClass().equals(key.getClass())))
{
> {code}
> I have managed to overcome this issue by adding the following null check:
> {code}
>         if (obj != null && method != null
>                 && objectClass.equals(obj.getClass())
>                 && (key == null || property != null && property.getClass().equals(key.getClass())))
{
> {code}
> I have only vague understanding of how things work inside jexl with tryInvoke() and unfortunately
can not provide any test case for this, but it has worked in my case, so if nobody have any
objections please include the fix in source tree.
> Thanks!



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message