]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
http2: disable asssertion blocking OSSFuzz testing
authorStefan Eissing <stefan@eissing.org>
Tue, 25 Jul 2023 15:14:21 +0000 (17:14 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Thu, 27 Jul 2023 15:44:15 +0000 (17:44 +0200)
- not clear how this triggers and it blocks OSSFuzz testing other
  things. Since we handle the case with an error return, disabling the
  assertion for now seems the best way forward.

Fixes #11500
Closes #11519

lib/http2.c

index 6c09ec1a1f75274d03608937cf0bc7c5beb84d42..0956e510c8bb1e923e90159a044882dadd6e0d99 100644 (file)
@@ -2077,7 +2077,9 @@ static ssize_t cf_h2_send(struct Curl_cfilter *cf, struct Curl_easy *data,
     else if(stream->upload_blocked_len) {
       /* the data in `buf` has alread been submitted or added to the
        * buffers, but have been EAGAINed on the last invocation. */
-      DEBUGASSERT(len >= stream->upload_blocked_len);
+      /* TODO: this assertion triggers in OSSFuzz runs and it is not
+       * clear why. Disable for now to let OSSFuzz continue its tests.
+      DEBUGASSERT(len >= stream->upload_blocked_len); */
       if(len < stream->upload_blocked_len) {
         /* Did we get called again with a smaller `len`? This should not
          * happend. We are not prepared to handle that. */