AlphaX/qortal
1
0
Fork 0
forked from Qortal/qortal
Code Issues Pull Requests Projects Releases Wiki Activity

Check for an empty string as well as null in a couple of places, so that deserializeSizedString() can be safely used.

Browse Source
This commit is contained in:
CalDescent 2022-01-10 19:42:21 +00:00
parent e3b45cac0a
commit ccde725d3b
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/qortal/controller/arbitrary/ArbitraryDataManager.java
View File

@ -395,7 +395,7 @@ public class ArbitraryDataManager extends Thread {
List<byte[]> signatures = arbitrarySignaturesMessage.getSignatures(); List<byte[]> signatures = arbitrarySignaturesMessage.getSignatures();
String peerAddress = peer.getPeerData().getAddress().toString(); String peerAddress = peer.getPeerData().getAddress().toString();
if (arbitrarySignaturesMessage.getPeerAddress() != null) { if (arbitrarySignaturesMessage.getPeerAddress() != null && !arbitrarySignaturesMessage.getPeerAddress().isEmpty()) {
// This message is about a different peer than the one that sent it // This message is about a different peer than the one that sent it
peerAddress = arbitrarySignaturesMessage.getPeerAddress(); peerAddress = arbitrarySignaturesMessage.getPeerAddress();
} }

2
src/main/java/org/qortal/network/Network.java
View File

@ -1109,7 +1109,7 @@ public class Network {
// External IP / peerAddress tracking // External IP / peerAddress tracking
public void ourPeerAddressUpdated(String peerAddress) { public void ourPeerAddressUpdated(String peerAddress) {
if (peerAddress == null) { if (peerAddress == null || peerAddress.isEmpty()) {
return; return;
} }