[TINKERPOP-2298] Bytecode.java flattenArguments throw exception when null

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.3.3
• Fix Version/s: 3.3.9, 3.4.4, 3.5.0
• Component/s: process
• Assignee: Stephen Mallette
• Reporter: Stark Arya
• Created: 20/Sep/19 06:58
• Updated: 30/Sep/19 10:56
• Resolved: 30/Sep/19 10:56

Description

In Bytecode.java,  current flattenArguments function  will trigger NPE exception when == null, which effect all version later from 3.3.3

private final Object[] flattenArguments(final Object... arguments)

Unknown macro: {     if (arguments.length == 0)           return EMPTY_ARRAY; }

 

Recurrent step：

1. with TinkerGraph.Mordern Graph

2. gremlin> g.V(1).outE(null)gremlin> g.V(1).outE(null)java.lang.NullPointerExceptionType ':help' or ':h' for help.Display stack trace? [yN]y
   java.lang.NullPointerException at org.apache.tinkerpop.gremlin.process.traversal.Bytecode.flattenArguments(Bytecode.java:265) at org.apache.tinkerpop.gremlin.process.traversal.Bytecode.addStep(Bytecode.java:84) at org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.outE(GraphTraversal.java:394) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

      3. but when has more arguments, it could produce results correctly

remlin> g.V(1).outE("knows", null)
==>e[7][1-knows->2]
==>e[8][1-knows->4]