[jira] [Comment Edited] (PHOENIX-3413) Ineffective null check in LiteralExpression#newConstant()

["Ted Yu (JIRA)" <ji...@apache.org>]

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

Ted Yu edited comment on PHOENIX-3413 at 3/13/17 1:12 AM:
----------------------------------------------------------

Don't know where the release audit came from.
Should be unrelated.


was (Author: yuzhihong@gmail.com):
Don't know where the release audit came from.

> Ineffective null check in LiteralExpression#newConstant()
> ---------------------------------------------------------
>
>                 Key: PHOENIX-3413
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3413
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.9.1
>            Reporter: Ted Yu
>            Assignee: Kevin Liew
>            Priority: Minor
>             Fix For: 4.11.0
>
>         Attachments: PHOENIX-3413.2.patch, PHOENIX-3413.3.patch, PHOENIX-3413.patch
>
>
> {code}
>         if (maxLength == null) {
>             maxLength = type == null || !type.isFixedWidth() ? null : type.getMaxLength(value);
>         }
> {code}
> The null check for type is ineffective - type is de-referenced in various places prior to the above check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)