ngtcp2: fix compiler warning about possible null-deref

- compiler analyzer did not include the call context for this
  static function where the condition had already been checked.
- eleminating the problem by making stream a call parameter

Fixes #11147
Closes #11151
This commit is contained in:
Stefan Eissing 2023-05-19 09:33:59 +02:00 committed by Daniel Stenberg
parent fbe23b5797
commit e0ddfc8e05
No known key found for this signature in database
GPG key ID: 5CC908FDB71E12C2

View file

@ -1312,13 +1312,12 @@ fail:
static ssize_t recv_closed_stream(struct Curl_cfilter *cf,
struct Curl_easy *data,
struct stream_ctx *stream,
CURLcode *err)
{
struct stream_ctx *stream = H3_STREAM_CTX(data);
ssize_t nread = -1;
(void)cf;
DEBUGASSERT(stream);
if(stream->reset) {
failf(data,
"HTTP/3 stream %" PRId64 " reset by server", stream->id);
@ -1413,7 +1412,7 @@ static ssize_t cf_ngtcp2_recv(struct Curl_cfilter *cf, struct Curl_easy *data,
}
else {
if(stream->closed) {
nread = recv_closed_stream(cf, data, err);
nread = recv_closed_stream(cf, data, stream, err);
goto out;
}
*err = CURLE_AGAIN;