-
Notifications
You must be signed in to change notification settings - Fork 54
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
Possible memory leak #344
Comments
Hi @fulalas - will look into this, you are using WhatsApp, right? |
There was indeed a couple of memory leaks (for whatsapp protocol), and the above commit should address them. Thanks for reporting this! 👍 |
Thanks a lot! It definitely helped, but there's still some memory leak: after a few hours the memory consumption went from 90 MB to 145 MB. Maybe you could use valgrind, no? |
I did it, and here's the initial result:
|
Ok thanks for checking. I admittedly only tested for ~1 hour and saw it was stable here.
I tried with valgrind but it didn't report any "non-constant" leaks, so I used a tool of my own heapusage and fixed the leaks it detected. I have yet to run a longer (and more active) session though. I'll re-open the issue until it's completely fixed. Btw, you are only using WhatsApp right, one account/profile? |
Sorry, I forgot to answer that. Yes, I am using both WhatsApp and Telegram. Only one account per protocol, yes. When I send a file (a simple image), even to myself, I can see the memory usage increasing. It happens in both Telegram and WhatsApp protocols. I did this a couple of times in both, and memory consumption jumped from 90 MB to 127 MB. |
Thanks, that is useful info 👍 Will continue look into it. |
Oh, 3 more things that I'm not sure if can be useful to inform:
|
Memory consumption goes from 90 MB to 187 MB after some time (hours) using it. Is there a reason for that?
Tested nchat 5.5.13 on Porteux 1.9
The text was updated successfully, but these errors were encountered: