Mercurial > hg > nginx

diff src/event/ngx_event_quic_transport.c @ 8342:76e8ec502c69 quic

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fixed error descriptions. The check for array bound is done inside function that returns error description. Missing initialization element is added.
author Vladimir Homutov <vl@nginx.com>
date Thu, 16 Apr 2020 16:54:22 +0300
parents 0f9e9786b90d
children e0abe17a2878
line wrap: on
line diff
--- a/src/event/ngx_event_quic_transport.c
+++ b/src/event/ngx_event_quic_transport.c
@@ -99,9 +99,10 @@ static char *ngx_quic_errors[] = {
     "CONNECTION_ID_LIMIT_ERROR",
     "PROTOCOL_VIOLATION",
     "INVALID_TOKEN",
-    "",
+    "unknown error 0xC",
     "CRYPTO_BUFFER_EXCEEDED",
-    "",
+    "unknown error 0xE",
+    "unknown error 0xF",
     "CRYPTO_ERROR",
 };
 
@@ -266,6 +267,11 @@ ngx_quic_build_int(u_char **pos, uint64_
 u_char *
 ngx_quic_error_text(uint64_t error_code)
 {
+
+    if (error_code >= NGX_QUIC_ERR_LAST) {
+        return (u_char *) "unknown error";
+    }
+
     return (u_char *) ngx_quic_errors[error_code];
 }
 
@@ -777,13 +783,6 @@ ngx_quic_parse_frame(ngx_quic_header_t *
 
         if (f->type == NGX_QUIC_FT_CONNECTION_CLOSE) {
 
-            if (f->u.close.error_code >= NGX_QUIC_ERR_LAST) {
-                ngx_log_error(NGX_LOG_INFO, pkt->log, 0,
-                              "unkown error code: %ui, truncated",
-                              f->u.close.error_code);
-                f->u.close.error_code = NGX_QUIC_ERR_LAST - 1;
-            }
-
             ngx_log_debug4(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
                           "CONN.CLOSE: { %s (0x%xi) type=0x%xi reason='%V'}",
                            ngx_quic_error_text(f->u.close.error_code),