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

C2DM Memory Leak #99

Closed
Runescrye opened this issue Feb 10, 2013 · 1 comment
Closed

C2DM Memory Leak #99

Runescrye opened this issue Feb 10, 2013 · 1 comment

Comments

@Runescrye
Copy link

Similar to issue #90, I am having more memory leak issues (Although far less frequently now after the GCM fix!).

Profiler seems to indicate it's an issue with the c2dm service where c2dms are piling.
I found a possible unclosed WebResponse at c2dmTransportAsync.cs line 242 - Same as the GCM really - an exception is thrown without a catch/finally to make sure the WebResponse is closed.

I am not sure if this is the only issue which is causing the problem, as I have far less C2DMs (only older clients) and the service can run for a week without a noticeable leak so it's hard to determine.

@Redth
Copy link
Owner

Redth commented Feb 11, 2013

Thanks, fixing this, hadn't updated it the same from GCM as you mentioned!

@Redth Redth closed this as completed in 14a0793 Feb 11, 2013
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

2 participants