[
https://issues.apache.org/jira/browse/DERBY-5465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Knut Anders Hatlen updated DERBY-5465:
--------------------------------------
Attachment: null.diff
Attaching a patch which replaces the broken equals() calls with ==.
> Broken null checks in NsTest
> ----------------------------
>
> Key: DERBY-5465
> URL: https://issues.apache.org/jira/browse/DERBY-5465
> Project: Derby
> Issue Type: Bug
> Components: Test
> Affects Versions: 10.8.2.2
> Reporter: Knut Anders Hatlen
> Assignee: Knut Anders Hatlen
> Priority: Minor
> Attachments: null.diff
>
>
> NsTest contains more occurrences of the broken null checking pattern found in
> DERBY-5452.
> Initializer.java:
> if (e.getMessage().equals(null)) {
> e.printStackTrace(System.out);
> }
> DbSetup.java:
> if (e.getMessage().equals(null)) {
> System.out.println("NULL error message detected");
> System.out.println("Here is the NULL exection - " +
> e.toString());
> System.out.println("Stack trace of the NULL exception -
> ");
> e.printStackTrace(System.out);
> }
> DbUtil.java:
> if (e.getMessage().equals(null)) {
> System.out.println(getThreadName()
> + " dbUtil --> NULL error message
> detected");
> System.out
> .println(getThreadName()
> + " dbUtil --> Here is the NULL
> exection - "
> + e.toString());
> System.out.println(getThreadName()
> + " dbUtil --> Stack trace of the NULL
> exception - ");
> e.printStackTrace(System.out);
> }
> If getMessage() returns null, the null checks will throw a
> NullPointerException instead of evaluating to true.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
