Skip to content

Commit

Permalink
Fix NPE if metrics are not set
Browse files Browse the repository at this point in the history
  • Loading branch information
Alemiz112 committed Jul 24, 2024
1 parent 8128edb commit 241c262
Showing 1 changed file with 10 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,10 @@ private void send(ChannelHandlerContext ctx, RakMessage message) {
throw new IllegalArgumentException();
}

getMetrics().encapsulatedOut(1);
RakChannelMetrics metrics = this.getMetrics();
if (metrics != null) {
metrics.encapsulatedOut(1);
}
EncapsulatedPacket[] packets = this.createEncapsulated(message);
if (message.priority() == RakPriority.IMMEDIATE) {
this.sendImmediate(ctx, packets);
Expand Down Expand Up @@ -317,14 +320,18 @@ private void handleDatagram(ChannelHandlerContext ctx, RakDatagramPacket packet)
// Not reassembled
continue;
}
getMetrics().encapsulatedIn(1);
if (metrics != null) {
metrics.encapsulatedIn(1);
}
try {
this.checkForOrdered(ctx, reassembled);
} finally {
reassembled.release();
}
} else {
getMetrics().encapsulatedIn(1);
if (metrics != null) {
metrics.encapsulatedIn(1);
}
this.checkForOrdered(ctx, encapsulated);
}
}
Expand Down

0 comments on commit 241c262

Please sign in to comment.