From d213bda09688eaee9a11466c2310aab7b82dee73 Mon Sep 17 00:00:00 2001
From: ghy <guo_hongyu@pku.edu.cn>
Date: Thu, 22 Apr 2021 10:57:24 +0800
Subject: [PATCH] code review: Transport

---
 src/main/java/logicface/Transport.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/main/java/logicface/Transport.java b/src/main/java/logicface/Transport.java
index 3a89bd7..84283ab 100644
--- a/src/main/java/logicface/Transport.java
+++ b/src/main/java/logicface/Transport.java
@@ -1,5 +1,6 @@
 package logicface;
 
+import common.LoggerHelper;
 import encoding.*;
 import packet.LpPacket;
 import packet.PacketException;
@@ -33,11 +34,13 @@ public class Transport {
     public LpPacket parseByteArray2LpPacket(byte[] buf) throws LogicFaceException {
         try {
             Block block=new Block(buf,true);
-            if(block.isValid()){
+            if(!block.isValid()){
+                LoggerHelper.logger.error("recv packet from face invalid");
                 return null;
             }
             LpPacket lpPacket=new LpPacket();
             if(!lpPacket.wireDecode(block)){
+                LoggerHelper.logger.error("parse to lpPacket error");
                 return null;
             }
             return lpPacket;