Merge pull request netty#934 from jpinner/fix_spdy_regression_master

SPDY: fix regression in SpdyFrameDecoder.decodeHeaderBlock()
jimmyinterview · Jan 14, 2013 · c20d875 · c20d875
2 parents 1354b4a + f75b6b1
commit c20d875
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyFrameDecoder.java b/codec-http/src/main/java/io/netty/handler/codec/spdy/SpdyFrameDecoder.java
@@ -578,7 +578,7 @@ private void decodeHeaderBlock(ByteBuf buffer) throws Exception {
             }
         }
 
-        while (numHeaders -- > 0) {
+        while (numHeaders > 0) {
             int headerSize = this.headerSize;
             decompressed.markReaderIndex();
 
@@ -638,6 +638,7 @@ private void decodeHeaderBlock(ByteBuf buffer) throws Exception {
                     return;
                 } else {
                     spdyHeaderBlock.addHeader(name, "");
+                    numHeaders --;
                     this.headerSize = headerSize;
                     continue;
                 }
@@ -683,6 +684,7 @@ private void decodeHeaderBlock(ByteBuf buffer) throws Exception {
                 index ++;
                 offset = index;
             }
+            numHeaders --;
             this.headerSize = headerSize;
         }
         decompressed = null;