You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Bill Havanki (JIRA)" <ji...@apache.org> on 2014/06/03 19:08:05 UTC

[jira] [Created] (ACCUMULO-2854) Bad null check in BulkImport.createNewBulkDir()

Bill Havanki created ACCUMULO-2854:
--------------------------------------

             Summary: Bad null check in BulkImport.createNewBulkDir()
                 Key: ACCUMULO-2854
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2854
             Project: Accumulo
          Issue Type: Bug
          Components: master
    Affects Versions: 1.6.0
            Reporter: Bill Havanki
             Fix For: 1.6.1


At the beginning of {{createNewBulkDir}} in {{o.a.a.master.tableOps.BulkImport}}:

{code}
    String tableDir = fs.matchingFileSystem(new Path(sourceDir), ServerConstants.getTablesDirs()).toString();
    
    if (tableDir == null)
      throw new IllegalStateException(sourceDir + " is not in a known namespace");
{code}

The call to {{matchingFileSystem()}} can return null, and if it does, a NullPointerException will be thrown. {{toString()}} on a non-null result will not return null, so the null check has no effect.



--
This message was sent by Atlassian JIRA
(v6.2#6252)