search

valyala / gozstd

go wrapper for zstd
MIT License
420 stars 60 forks source link

Fix memory leaks in writer test case #31

Closed guillemj closed 3 years ago

guillemj commented 3 years ago

This causes the test to fail on 32-bit systems, as the system exhausts its available address space due to the leaks.

An example failure https://ci.debian.net/data/autopkgtest/testing/i386/g/golang-github-valyala-gozstd/9646307/log.gz.

codecov[bot] commented 3 years ago

Codecov Report

Merging #31 (d7fc976) into master (cc59ecd) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   94.90%   94.90%           
=======================================
  Files           5        5           
  Lines         510      510           
=======================================
  Hits          484      484           
  Misses         16       16           
  Partials       10       10

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update cc59ecd...d7fc976. Read the comment docs.

valyala commented 3 years ago

@guillemj , thanks for the contribution!