directory-kerby mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yan, Yan A" <yan.a....@intel.com>
Subject RE: [jira] [Updated] (DIRKRB-408) Fix findbugs issues
Date Thu, 07 Jan 2016 00:46:14 GMT
Hi Kai,

I think the remaining bugs in DIRKRB-408 will not make much trouble at the moment. 

However, there are several bugs in other modules may cause unexpected exceptions,
Such as null pointer dereference, default encoding and resource clean up.
I have opened two more JIRA:
https://issues.apache.org/jira/browse/DIRKRB-520
https://issues.apache.org/jira/browse/DIRKRB-521
And I will try to find the most important ones to fix. 

I will quickly finish them this morning. 
And I hope I could help you with more important things about Kerby after that.

Best regards,
Yan


-----Original Message-----
From: Zheng, Kai [mailto:kai.zheng@intel.com] 
Sent: Wednesday, January 06, 2016 5:47 PM
To: kerby@directory.apache.org
Subject: FW: [jira] [Updated] (DIRKRB-408) Fix findbugs issues

Hi Yan,

Can you browse the reported findbug issues and pick up the DOES-MAKE-SENSE ones to fix?
As we have more important things to do, I would suggest we focus on the important things.

Kerby won't pretend to be perfect in current  phase, but does want to make more sense to users.

Thanks.

Regards,
Kai

-----Original Message-----
From: YanYan (JIRA) [mailto:jira@apache.org] 
Sent: Wednesday, January 06, 2016 5:21 PM
To: dev@directory.apache.org
Subject: [jira] [Updated] (DIRKRB-408) Fix findbugs issues


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

YanYan updated DIRKRB-408:
--------------------------
    Attachment: DIRKRB-408.patch

Fix most bugs in the 5 modules in Description. However, there are 2 remaining bugs:
{code}
Unused field: org.apache.kerby.benchmark.generated.JsonBackendBenchmark_jmhType_B1.p000
{code}
In fact, there are 2304 of this same kind of bugs.
{code}
new org.apache.kerby.kerberos.kerb.integration.test.Transport$Message(String, byte[]) may
expose internal representation by storing an externally mutable object into Transport$Message.body
{code} I have tried {{this.body = body.clone()}}, however, it makes an error when test.
Pulling request link: [https://github.com/apache/directory-kerby/pull/5]

> Fix findbugs issues
> --------------------
>
>                 Key: DIRKRB-408
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-408
>             Project: Directory Kerberos
>          Issue Type: Bug
>    Affects Versions: 1.0.0-RC2
>            Reporter: Xu Yaning
>            Assignee: YanYan
>         Attachments: DIRKRB-408.patch
>
>
> When the source code is built with findbugs, there are some bugs, mainly in benchmark
module, json-backend module, integration-test module, kerb-util module and kdc-tool module.



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