Merge pull request #126 from expressvpn/CVPN-1585-no-data-frames-befo…

…re-online

lightway-core: Ignore `wire::Frame::Data{Frag}` until `State::Online`

lightway-core/src/connection.rs

@@ -1334,13 +1334,21 @@ impl<AppState: Send> Connection<AppState> {
     }
 
     fn handle_outside_data_packet(&mut self, data: wire::Data) -> ConnectionResult<()> {
+        if !matches!(self.state, State::Online) {
+            return Err(ConnectionError::InvalidState);
+        }
+
         // into_owned should be a NOP here since
         // `wire::Data::try_from_wire` produced a `Cow::Owned`
         // variant.
         self.handle_outside_data_bytes(data.data.into_owned())
     }
 
     fn handle_outside_data_fragment(&mut self, frag: wire::DataFrag) -> ConnectionResult<()> {
+        if !matches!(self.state, State::Online) {
+            return Err(ConnectionError::InvalidState);
+        }
+
         match self.fragment_map.add_fragment(frag) {
             FragmentMapResult::Complete(data) => {
                 self.handle_outside_data_bytes(data)?;