[jira] [Reopened] (ACCUMULO-2870) Potential null pointer dereference in Tablet#longestCommonLength()

["Josh Elser (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-2870?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser reopened ACCUMULO-2870:
----------------------------------


Ugh, this is why I shouldn't try to JIRA when under the weather. Didn't commit this yet. Wanted to try to add a test.

> Potential null pointer dereference in Tablet#longestCommonLength()
> ------------------------------------------------------------------
>
>                 Key: ACCUMULO-2870
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2870
>             Project: Accumulo
>          Issue Type: Bug
>    Affects Versions: 1.5.1, 1.6.0
>            Reporter: Ted Yu
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.5.2, 1.6.1, 1.7.0
>
>         Attachments: 0001-ACCUMULO-2870-Fail-hard-and-fast-if-we-don-t-calcula.patch
>
>
> longestCommonLength() is called by findSplitRow():
> {code}
>       Text text = (mid == null) ? null : mid.getRow();
>       SortedMap<Double,Key> firstHalf = keys.headMap(.5);
>       if (firstHalf.size() > 0) {
>         Text beforeMid = firstHalf.get(firstHalf.lastKey()).getRow();
>         Text shorter = new Text();
>         int trunc = longestCommonLength(text, beforeMid);
> {code}
> If mid is null, text would be null.
> However, text is dereferenced without null check in longestCommonLength().



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