diff options
author | andreast <andreast@138bc75d-0d04-0410-961f-82ee72b054a4> | 2004-09-21 15:30:20 +0000 |
---|---|---|
committer | andreast <andreast@138bc75d-0d04-0410-961f-82ee72b054a4> | 2004-09-21 15:30:20 +0000 |
commit | 2bbaf593cf931926f7464a5c76dc80594e7d2815 (patch) | |
tree | bc309268f61548dfd2fd03375ad799b9964d2e6e /libjava/java/security | |
parent | 62e8d0c9c16b08f9adf9649b5e72cb13ea2bbdf6 (diff) | |
download | ppe42-gcc-2bbaf593cf931926f7464a5c76dc80594e7d2815.tar.gz ppe42-gcc-2bbaf593cf931926f7464a5c76dc80594e7d2815.zip |
2004-09-21 Casey Marshall <csm@gnu.org>
* java/security/cert/X509CRLSelector.java:
(match): remove unreachable try-catch clauses.
Reported by: Dalibor Topic <robilad@kaffe.org>
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@87805 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'libjava/java/security')
-rw-r--r-- | libjava/java/security/cert/X509CRLSelector.java | 56 |
1 files changed, 21 insertions, 35 deletions
diff --git a/libjava/java/security/cert/X509CRLSelector.java b/libjava/java/security/cert/X509CRLSelector.java index 3264e469e2e..a59791e6759 100644 --- a/libjava/java/security/cert/X509CRLSelector.java +++ b/libjava/java/security/cert/X509CRLSelector.java @@ -378,8 +378,27 @@ public class X509CRLSelector implements CRLSelector, Cloneable BigInteger crlNumber = null; if (maxCrlNumber != null) { + byte[] b = crl.getExtensionValue(CRL_NUMBER_ID); + if (b == null) + return false; try { + DERValue val = DERReader.read(b); + if (!(val.getValue() instanceof BigInteger)) + return false; + crlNumber = (BigInteger) val.getValue(); + } + catch (IOException ioe) + { + return false; + } + if (maxCrlNumber.compareTo(crlNumber) < 0) + return false; + } + if (minCrlNumber != null) + { + if (crlNumber == null) + { byte[] b = crl.getExtensionValue(CRL_NUMBER_ID); if (b == null) return false; @@ -394,42 +413,9 @@ public class X509CRLSelector implements CRLSelector, Cloneable { return false; } - if (maxCrlNumber.compareTo(crlNumber) < 0) - return false; - } - catch (CertificateParsingException cpe) - { - return false; - } - } - if (minCrlNumber != null) - { - try - { - if (crlNumber == null) - { - byte[] b = crl.getExtensionValue(CRL_NUMBER_ID); - if (b == null) - return false; - try - { - DERValue val = DERReader.read(b); - if (!(val.getValue() instanceof BigInteger)) - return false; - crlNumber = (BigInteger) val.getValue(); - } - catch (IOException ioe) - { - return false; - } - } - if (minCrlNumber.compareTo(crlNumber) > 0) - return false; - } - catch (CertificateParsingException cpe) - { - return false; } + if (minCrlNumber.compareTo(crlNumber) > 0) + return false; } if (date != null) { |