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

[Windows] Split Tunneling: uploading relatively big site list causes memory leak #1219

Open
Rebel028 opened this issue Oct 30, 2024 · 0 comments

Comments

@Rebel028
Copy link

Describe the bug
Uploading relatively big site list causes memory leak. After restarting the app it's impossible to reset the list because it requires to load the whole list into the UI (that causes memory leak again). Mentioned in #1135 but was not solved

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Settings'->'Connection'->'Site-based split tunneling'
  2. Click on three dots then 'import'
  3. Choose file for import: amnezia_sites.json (this file is very big but you can reproduce in with file from Client hangs and crashes on boot when there are too many sites in the Split tunneling site list #1135)
  4. Application starts consuming RAM like crazy

Expected behavior
List is loaded successfully and app is usable

Log files
Logs are enabled in settings, but no logs are written

Desktop (please complete the following information):

  • OS: Windows 11
  • Version 4.8.1.9

Additional context

Related issues:

It probably has something to do with client/ui/controllers/sitesController.cpp and I would gladly fixed it myself if I'd ever worked with Qt and C++

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

No branches or pull requests

1 participant