http2: Fix bug that associated stream canceled on PUSH_PROMISE

Previously we don't ignore PUSH_PROMISE header fields in on_header callback. It makes header values mixed with following HEADERS, resulting protocol error.
curl · Feb 9, 2015 · 7eebf9a · 7eebf9a
1 parent 20c727e
commit 7eebf9a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/lib/http2.c b/lib/http2.c
@@ -424,6 +424,11 @@ static int on_header(nghttp2_session *session, const nghttp2_frame *frame,
   (void)frame;
   (void)flags;
 
+  /* Ignore PUSH_PROMISE for now */
+  if(frame->hd.type != NGHTTP2_HEADERS) {
+    return 0;
+  }
+
   if(frame->hd.stream_id != c->stream_id) {
     return 0;
   }