ngtcp2: Reset dynbuf when it is fully drained

Reported-by: vl409 on github
Fixes #7351
Closes #8504

diff --git a/docs/KNOWN_BUGS b/docs/KNOWN_BUGS
index 39a2995e6d5bf79d41bb3eb32874d31c12e8adad..3ed5d5a1374820df51d72e134786e4f43b11e557 100644 (file)
--- a/docs/KNOWN_BUGS
+++ b/docs/KNOWN_BUGS
@@ -159,7 +159,6 @@ problems may have been fixed or changed somewhat since this was written.
  18. HTTP/3
  18.1 If the HTTP/3 server closes connection during upload curl hangs
  18.2 Uploading HTTP/3 files gets interrupted at certain file sizes
- 18.4 Downloading with HTTP/3 produces broken files
  18.6 HTTP/3 multipart POST with quiche fails
  18.8 HTTP/3 does not support client certs
  18.9 connection migration does not work
@@ -1126,10 +1125,6 @@ problems may have been fixed or changed somewhat since this was written.
 
  See https://github.com/curl/curl/issues/6510
 
-18.4 Downloading with HTTP/3 produces broken files
-
- See https://github.com/curl/curl/issues/7351
-
 18.6 HTTP/3 multipart POST with quiche fails
 
  https://github.com/curl/curl/issues/7125

diff --git a/lib/vquic/ngtcp2.c b/lib/vquic/ngtcp2.c
index 8c7fb81e34d3cd0b3b80fca784734da94fa75d73..aac092b8d2c0674ef77e94f253c06849cf9a321a 100644 (file)
--- a/lib/vquic/ngtcp2.c
+++ b/lib/vquic/ngtcp2.c
@@ -1220,6 +1220,8 @@ static size_t drain_overflow_buffer(struct HTTP *stream)
     if(ncopy != overlen)
       /* make the buffer only keep the tail */
       (void)Curl_dyn_tail(&stream->overflow, overlen - ncopy);
+    else
+      Curl_dyn_reset(&stream->overflow);
   }
   return ncopy;
 }