accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1361) VisibilityEvaluator throws NPE when passed a zero length byte array
Date Tue, 30 Apr 2013 03:30:15 GMT

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

Josh Elser commented on ACCUMULO-1361:
--------------------------------------

IMO, as long as the VisibilityEvaluator exists in the codebase (maybe aside from being private),
easy fixes are a worthwhile investment. I think (please correct me if I'm wrong) [~ctubbsii]
was more noting that this isn't a bug in Accumulo as you described it. It's an improvement
as it makes the "public API" better.
                
> VisibilityEvaluator throws NPE when passed a zero length byte array
> -------------------------------------------------------------------
>
>                 Key: ACCUMULO-1361
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1361
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Chris McCubbin
>            Assignee: Corey J. Nolet
>            Priority: Minor
>             Fix For: 1.6.0
>
>
> The following code snippet throws a NPE on a fairly recent version of 1.5. I would expect
it to return true, or at the very least, to throw a meaningful exception.
> {code:title=SadPanda.java|borderStyle=solid}
> VisibilityEvaluator evaluator = new VisibilityEvaluator(new Authorizations("a".getBytes()));
> ColumnVisibility cv = new ColumnVisibility(new byte[0]);
> //The following line throws a NPE
> boolean result = evaluator.evaluate(cv);
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message