[JEXL-239] Add NPE check to property for MapGetExecutor

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1.1
• Fix Version/s: 3.0
• Flags: Patch
• Assignee: Henri Biestro
• Reporter: Bruno P. Kinoshita
• Created: 06/Sep/17 04:10
• Updated: 29/Jun/21 06:59
• Resolved: 06/Oct/17 14:53

Description

From GitHub pull request #2 https://github.com/apache/commons-jexl/pull/2

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.

Issue Links

• is related to: JEXL-221 Sporadic undefined property error caused by NPE at MapGetExecutor.tryInvoke()
• links to: Pull request #2