You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by "Xu Yaning (JIRA)" <ji...@apache.org> on 2015/07/13 07:21:05 UTC

[jira] [Updated] (DIRKRB-354) Fix getBooleanOption return null problem

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

Xu Yaning updated DIRKRB-354:
-----------------------------
    Attachment: DIRKRB-354-v1.patch

In this patch, I disable {{getBooleanOption(KOption option)}} to return {{null}} by passing a parameter {{defaultValue}} such that it becomes {{getBooleanOption(KOption option, Boolean defaultValue)}}, and modify the codes where invoke it. Pls help me review it, thank you~

> Fix getBooleanOption return null problem
> ----------------------------------------
>
>                 Key: DIRKRB-354
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-354
>             Project: Directory Kerberos
>          Issue Type: Bug
>            Reporter: Xu Yaning
>         Attachments: DIRKRB-354-v1.patch
>
>
> org.apache.kerby.KOptions.getBooleanOption(KOption) has Boolean return type and returns explicit null; A method that returns either Boolean.TRUE, Boolean.FALSE or null is an accident waiting to happen. This method can be invoked as though it returned a value of type boolean, and the compiler will insert automatic unboxing of the Boolean value. If a null value is returned, this will result in a NullPointerException.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)