Mercurial > hg > nginx

diff src/http/v2/ngx_http_v2.c @ 7916:29795b697e14

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
HTTP/2: avoid memcpy() with NULL source and zero length. Prodded by Clang Static Analyzer.
author Maxim Dounin <mdounin@mdounin.ru>
date Tue, 31 Aug 2021 16:44:13 +0300
parents 9cf043a5d9ca
children 2245324a507a
line wrap: on
line diff
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -4241,7 +4241,9 @@ ngx_http_v2_process_request_body(ngx_htt
                 n = size;
             }
 
-            rb->buf->last = ngx_cpymem(rb->buf->last, pos, n);
+            if (n > 0) {
+                rb->buf->last = ngx_cpymem(rb->buf->last, pos, n);
+            }
 
             ngx_log_debug1(NGX_LOG_DEBUG_HTTP, fc->log, 0,
                            "http2 request body recv %uz", n);