Mercury-IM/Smack
Mercury-IM/Smack
1
0
Fork 0
mirror of https://codeberg.org/Mercury-IM/Smack synced 2024-11-15 20:12:04 +01:00
Code Issues Releases Wiki Activity

Assert that processPacket is not called with null

Browse source
This commit is contained in:
Florian Schmaus 2014-12-28 18:05:04 +01:00
parent d295939285
commit 5d5e3c05c5
2 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
smack-core/src/main/java/org/jivesoftware/smack/AbstractXMPPConnection.java
View file

@ -905,6 +905,7 @@ public abstract class AbstractXMPPConnection implements XMPPConnection {
* @param packet the packet to process. * @param packet the packet to process.
*/ */
protected void processPacket(Packet packet) { protected void processPacket(Packet packet) {
assert(packet != null);
lastStanzaReceived = System.currentTimeMillis(); lastStanzaReceived = System.currentTimeMillis();
// Deliver the incoming packet to listeners. // Deliver the incoming packet to listeners.
executorService.submit(new ListenerNotification(packet)); executorService.submit(new ListenerNotification(packet));

4
smack-core/src/main/java/org/jivesoftware/smack/PacketCollector.java
View file

@ -225,10 +225,6 @@ public class PacketCollector {
* @param packet the packet to process. * @param packet the packet to process.
*/ */
protected void processPacket(Packet packet) { protected void processPacket(Packet packet) {
if (packet == null) {
return;
}
if (packetFilter == null || packetFilter.accept(packet)) { if (packetFilter == null || packetFilter.accept(packet)) {
while (!resultQueue.offer(packet)) { while (!resultQueue.offer(packet)) {
// Since we know the queue is full, this poll should never actually block. // Since we know the queue is full, this poll should never actually block.