bodgit / sevenzip / 37257868943

78%

push

github

fix: close pooled readers instead of leaking them (#500)

Folder readers kept for reuse by File.Open could be leaked without
being closed, holding on to decompressor state such as LZMA
dictionaries, zstd decoder goroutines, and decoders that are only
returned to their sync.Pool on Close:

* Pool.Put silently dropped a reader if one was already pooled at the
  same offset.
* ReadCloser.Close never closed the pooled readers.
* A file closed after its archive returned its reader to the pool.
* File.Open didn't close a new folder reader if the initial Seek
  failed.

Close the reader in Pool.Put if it can't be pooled, add a Close method
to Pooler that closes all pooled readers and closes any later returned
ones immediately, call it from ReadCloser.Close before closing the
volumes, and close the reader on a failed Seek in File.Open.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

28 of 29 new or added lines in 2 files covered. (96.55%)

1965 of 2559 relevant lines covered (76.79%)

0.87 hits per line