Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory leak fix in #22 is hacky and should be fixed in the go-clamd dependency #27

Open
vjt opened this issue Jun 9, 2021 · 3 comments
Assignees
Labels

Comments

@vjt
Copy link
Contributor

vjt commented Jun 9, 2021

As described and fixed in #22 - however the fix is hacky and should be proper handled in go-clamd.

@vjt vjt added the bug label Jun 9, 2021
@vjt vjt self-assigned this Jun 9, 2021
@Helios-23
Copy link

Any update on this? we have a fwe systems where even with 16gb of ram sometimes we get clamav crashes from memory use

@vjt
Copy link
Contributor Author

vjt commented Jun 21, 2021

Hi @Helios-23, sorry for the delay. As I wasn't yet able to focus on this, as @jblackman advised that this fix is good enough if we are pressed for time, I've merged #22 and released v0.7.3.

Please let me know how it goes. I'll rename this issue and keep this open as I prefer this is fixed properly anyway.

Thanks!

@vjt vjt changed the title Memory leak after many scan executions Memory leak fix in #22 is hacky and should be fixed in the go-clamd dependency Jun 21, 2021
@vjt vjt mentioned this issue Jun 21, 2021
@Helios-23
Copy link

Helios-23 commented Jun 22, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants