accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2870) Potential null pointer dereference in Tablet#longestCommonLength()
Date Sat, 07 Jun 2014 17:27:01 GMT

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

Christopher Tubbs commented on ACCUMULO-2870:
---------------------------------------------

This issue needs an affectedVersion.

> Potential null pointer dereference in Tablet#longestCommonLength()
> ------------------------------------------------------------------
>
>                 Key: ACCUMULO-2870
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2870
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Priority: Minor
>
> 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)

Mime
View raw message