commit 4d0f4bc1de5cde645a1aabffdbaacac1f955175c
author Piotr Sikora <piotrsikora@google.com> Wed Mar 15 14:45:39 2017 -0700
committer Piotr Sikora <piotrsikora@google.com> Sun Mar 26 08:55:59 2017 +0000
tree 06c26af08553aa09d0d44151b65d1a91f8a360a1
parent a5c90def0f5a7dd6d2b254f3844796d53ed56e1f

HTTP/2: emit PROTOCOL_ERROR on padding errors.

Change-Id: I29b41d6f52a53fda375abaa64c6ed8d4ba46e616
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Reviewed-on: https://nginx-review.googlesource.com/2823
Reviewed-by: Lizan Zhou <zlizan@google.com>

src/http/v2/ngx_http_v2.c

diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c
index 2b16ccb..e1d0d4a 100644
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -812,7 +812,8 @@
                           "with incorrect length: %uz, padding: %uz",
                           size, h2c->state.padding);
 
-            return ngx_http_v2_connection_error(h2c, NGX_HTTP_V2_SIZE_ERROR);
+            return ngx_http_v2_connection_error(h2c,
+                                                NGX_HTTP_V2_PROTOCOL_ERROR);
         }
 
         h2c->state.length -= h2c->state.padding + 1;
@@ -1055,7 +1056,8 @@
                           "with incorrect length: %uz, padding: %uz",
                           h2c->state.length, h2c->state.padding);
 
-            return ngx_http_v2_connection_error(h2c, NGX_HTTP_V2_SIZE_ERROR);
+            return ngx_http_v2_connection_error(h2c,
+                                                NGX_HTTP_V2_PROTOCOL_ERROR);
         }
 
         h2c->state.length -= h2c->state.padding;