Skip to content

fixed NullPointerException for arraycolls with null elements#9

Open
dnet wants to merge 1 commit intounsynchronized:masterfrom
dnet:arraycoll-npe-fix
Open

fixed NullPointerException for arraycolls with null elements#9
dnet wants to merge 1 commit intounsynchronized:masterfrom
dnet:arraycoll-npe-fix

Conversation

@dnet
Copy link

@dnet dnet commented Nov 27, 2015

If an arraycoll had a null element, the software crashed like below

Exception in thread "main" java.lang.NullPointerException
        at org.unsynchronized.arraycoll.toString(arraycoll.java:45)
        at org.unsynchronized.arrayobj.toString(arrayobj.java:31)
        at org.unsynchronized.jdeserialize.dump_Instance(jdeserialize.java:329)
        at org.unsynchronized.jdeserialize.dump(jdeserialize.java:961)
        at org.unsynchronized.jdeserialize.main(jdeserialize.java:1187)

This pull request fixes this, the string null is printed instead of calling .toString() on (and thus dereferencing) null object references.

@dnet
Copy link
Author

dnet commented Jan 30, 2017

Is there an issue with this PR? What's the reason behind not merging it?

@dnet
Copy link
Author

dnet commented Nov 6, 2017

Ping?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant