tealeg / xlsx

Go library for reading and writing XLSX files.
Other
5.81k stars 810 forks source link

fix: memory leak risk when file write err #744

Closed echoface closed 2 years ago

echoface commented 2 years ago

Here a risk memory leak without closing file descriptor correctly, when disk write fail.

ghost commented 2 years ago
👇 Click on the image for a new way to code review - Make big changes easier — review code in small groups of related files - Know where to start — see the whole change at a glance - Take a code tour — explore the change with an interactive tour - Make comments and review — all fully sync’ed with github [Try it now!](https://app.codesee.io/r/reviews?pr=744&src=https%3A%2F%2Fgithub.com%2Ftealeg%2Fxlsx)

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend