fix(protocol): Don't reserve memory on packet decoding error (#360)

* fix(protocol): Don't reserve memory on packet decoding error

* fix: revert packet_decoder fix part, this need more test for the moment

* fix: drop the connection on decoding error to avoid possible attack

pumpkin/src/client/mod.rs

@@ -503,7 +503,11 @@ impl Client {
                     return true;
                 }
                 Ok(None) => (), //log::debug!("Waiting for more data to complete packet..."),
-                Err(err) => log::warn!("Failed to decode packet for: {}", err.to_string()),
+                Err(err) => {
+                    log::warn!("Failed to decode packet for: {}", err.to_string());
+                    self.close();
+                    return false; // return to avoid reserving additional bytes
+                }
             }
 
             dec.reserve(4096);