[GitHub] trafodion pull request #1380: [TRAFODION-2886] fix the nullpointer error sca...

[xiaozhongwang]

Github user xiaozhongwang commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1380#discussion_r160868578
  
    --- Diff: core/sql/executor/cluster.cpp ---
    @@ -2395,7 +2395,7 @@ NABoolean Cluster::checkAndSplit(ExeErrorCode * rc)
                                         rc);
       
       if ( !next_ || *rc ) {
    -    if ( ! rc ) * rc = EXE_NO_MEM_TO_EXEC;
    +    * rc = EXE_NO_MEM_TO_EXEC;
    --- End diff --
    
    if rc is NULL,  core at line 2397
    so rc must be not NULL.
    
    or
    make sure rc is not NULL at line 2397



---