search

alibaba / xquic

XQUIC Library released by Alibaba is a cross-platform implementation of QUIC and HTTP/3 protocol.
Apache License 2.0
1.7k stars 327 forks source link

[Bug]: Is there any serious test on xquic? #265

Closed Luffbee closed 1 year ago

Luffbee commented 1 year ago

What happened?

I found many memory leak bugs, it seems xquic is not tested in extreme scenarios (e.g. high loss rate).

Steps To Reproduce

See my issues.

Relevant log output

No response

Yanmei-Liu commented 1 year ago

Actually we have tested in extreme scenarios (e.g 20 percent loss ratio). I think the problem you found is that the packet memory is cached in the queue under xqc_connection, which is reused many times but not released until the connection resource is freed. It's a lazy release mechanism considering performance. By the way, we limited the maximum packet queue length per connection.