Re: Patch fixing JDK-8176553

[Alan Bateman]

On 04/04/2018 15:06, David Lloyd wrote:

Actually I've talked to Jan and we're going to try again with the more
correct subject line & RFR.  Thanks.


It would be good to see if a test can be developed too.

-Alan

Re: Patch fixing JDK-8176553

[David Lloyd]

Actually I've talked to Jan and we're going to try again with the more
correct subject line & RFR.  Thanks.

On Wed, Apr 4, 2018 at 8:51 AM, David Lloyd  wrote:
> Is there anyone who would be willing to sponsor this change?
>
> On Tue, Jan 23, 2018 at 4:58 PM, Jan Kalina  wrote:
>> Hi,
>> I has prepared trivial patch for bug JDK-8176553, which I has reported.
>>
>> I will welcome if it could be merged into JDK.
>> (The bug is present in JDK9 and JDK8 too.)
>> I am covered by Red Hat OCA.
>>
>> The patch is attached, bug reproducer is already in JIRA:
>> https://bugs.openjdk.java.net/browse/JDK-8176553
>>
>> Thanks,
>> Jan Kalina
>
>
>
> --
> - DML



-- 
- DML

Re: Patch fixing JDK-8176553

[David Lloyd]

Is there anyone who would be willing to sponsor this change?

On Tue, Jan 23, 2018 at 4:58 PM, Jan Kalina  wrote:
> Hi,
> I has prepared trivial patch for bug JDK-8176553, which I has reported.
>
> I will welcome if it could be merged into JDK.
> (The bug is present in JDK9 and JDK8 too.)
> I am covered by Red Hat OCA.
>
> The patch is attached, bug reproducer is already in JIRA:
> https://bugs.openjdk.java.net/browse/JDK-8176553
>
> Thanks,
> Jan Kalina



-- 
- DML

Patch fixing JDK-8176553

[Jan Kalina]

Hi,
I has prepared trivial patch for bug JDK-8176553, which I has reported.

I will welcome if it could be merged into JDK.
(The bug is present in JDK9 and JDK8 too.)
I am covered by Red Hat OCA.

The patch is attached, bug reproducer is already in JIRA:
https://bugs.openjdk.java.net/browse/JDK-8176553

Thanks,
Jan Kalina
diff --git a/src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java b/src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java
--- a/src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java
+++ b/src/java.naming/share/classes/com/sun/jndi/ldap/AbstractLdapNamingEnumeration.java
@@ -312,7 +312,8 @@
 
 if ((refEx != null) &&
 (refEx.hasMoreReferrals() ||
- refEx.hasMoreReferralExceptions())) {
+ refEx.hasMoreReferralExceptions()) &&
+ ! (errEx instanceof LimitExceededException)) {
 
 if (homeCtx.handleReferrals == LdapClient.LDAP_REF_THROW) {
 throw (NamingException)(refEx.fillInStackTrace());