<Swing Dev> Integrated: 6514600: AbstractAction can throw NullPointerException when clone()d

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Tue Sep 29 08:23:00 UTC 2020


On Mon, 28 Sep 2020 07:52:20 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

> Please review a fix for an issue whereby it is seen that AbstractAction which has had its values clear()ed or nulled
> fails to clone and throws NPE. This is because AbstractAction contains an inner class ArrayTable, whose 'table' field
> is lazily instantiated, and nulled on clear(), and thus can be null. The clone() method assumes it is not null.
> Proposed fix is to add a check for null for "table" field in clone().

This pull request has now been integrated.

Changeset: 55c90a17
Author:    Prasanta Sadhukhan <psadhukhan at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/55c90a17
Stats:     70 lines in 2 files changed: 59 ins; 0 del; 11 mod

6514600: AbstractAction can throw NullPointerException when clone()d

Reviewed-by: serb, pbansal

-------------

PR: https://git.openjdk.java.net/jdk/pull/377


More information about the swing-dev mailing list