prevent a leak of 'active streams' if client request has user error (#266)

src/proto/streams/streams.rs

@@ -607,13 +607,21 @@ where
 
             let mut stream = me.store.insert(stream.id, stream);
 
-            me.actions.send.send_headers(
+            let sent = me.actions.send.send_headers(
                 headers,
                 send_buffer,
                 &mut stream,
                 &mut me.counts,
                 &mut me.actions.task,
-            )?;
+            );
+
+            // send_headers can return a UserError, if it does,
+            // we should forget about this stream.
+            if let Err(err) = sent {
+                stream.unlink();
+                stream.remove();
+                return Err(err.into());
+            }
 
             // Given that the stream has been initialized, it should not be in the
             // closed state.

tests/client_request.rs

@@ -440,9 +440,14 @@ fn request_with_connection_headers() {
     let _ = ::env_logger::try_init();
     let (io, srv) = mock::new();
 
-    let srv = srv.assert_client_handshake()
+    // can't assert full handshake, since client never sends a request, and
+    // thus never bothers to ack the settings...
+    let srv = srv.read_preface()
         .unwrap()
-        .recv_settings()
+        .recv_frame(frames::settings())
+        // goaway is required to make sure the connection closes because
+        // of no active streams
+        .recv_frame(frames::go_away(0))
         .close();
 
     let headers = vec![