Hi Jaroslav, This looks reasonable. I assume you have run the JCK to verify that it doesn't break anything else?
best regards, -- daniel On 7/11/13 1:48 PM, Jaroslav Bachorik wrote:
Please, review the change. http://cr.openjdk.java.net/~jbachorik/8019584/webrev.00/ The combination of the fix for JDK-8014085 and ObjectInputStream.readFields() not throwing CNFE when trying to deserialize an object graph containing references to non-available classes makes an InvalidObjectException being thrown instead of the CNFE when processing JMX notifications. The patch makes the ClientNotificationForwarder ready for InvalidObjectException - it will correctly report lost notifications but will not cause the notification processing loop to fail with unhandled exception. Thanks, -JB-