[jira] Resolved: (HARMONY-2487) [classlib][security] java.security.cert.X509CertSelector.addSubjectAlternativeName(int, null) throws IOException while RI throws NullPointerException (RI compatibility issue)

["Stepan Mishura (JIRA)" <ji...@apache.org>]

     [ http://issues.apache.org/jira/browse/HARMONY-2487?page=all ]

Stepan Mishura resolved HARMONY-2487.
-------------------------------------

    Resolution: Fixed

Thanks Anton - the bug was fixed in SECURITY module at r483841.

Please check that the fix fully resolves your problem.

> [classlib][security] java.security.cert.X509CertSelector.addSubjectAlternativeName(int, null) throws IOException while RI throws NullPointerException (RI compatibility issue)
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-2487
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2487
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Anton Ivanov
>         Assigned To: Stepan Mishura
>            Priority: Trivial
>
> IOException is thrown for java.security.cert.X509CertSelector.addSubjectAlternativeName(int type, byte[] name) when name is null while RI throws NullPointerException.
> There is no mention in specification what exception should be thrown when parameter name is null.
> It's a compatibility issue.
> The same problem is with java.security.cert.X509CertSelector.addPathToName(int type, byte[] name).
> Test to reproduce the problem:
> import junit.framework.TestCase;
> import java.security.cert.X509CertSelector; 
> import java.io.IOException; 
> public class X509CertSelectorTest extends TestCase {
>         
>     public void test1() { 
>         int type = 0;
>         X509CertSelector sl = new X509CertSelector();
>         
>         try {
>             sl.addSubjectAlternativeName(type,(byte[])null); 
>         } catch (NullPointerException e) {
>             //passed
>         } catch (Exception e) {
>             fail("NullPointerException should be thrown");
>         }
>     }
>     public void test2() { 
>         int type = 0;
>         X509CertSelector sl = new X509CertSelector();
>         
>         try {
>             sl.addPathToName(type,(byte[])null); 
>         } catch (NullPointerException e) {
>             //passed
>         } catch (Exception e) {
>             fail("NullPointerException should be thrown");
>         }
>     }
> } 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira