What are the most common (and often overlooked) causes of memory leaks in managed (.net) applications?

后端 未结 11 964
闹比i
闹比i 2021-02-04 09:32

Please can anyone recommend a quick checklist / best practice guide to help us avoid simple (but subtle) mistakes that cause could cause memory leaks in .net apps

I find

相关标签:
11条回答
  • 2021-02-04 10:00

    There are many forms of leaks:

    • Unmanaged leaks (code that allocates unmanaged code)
    • Resource leaks (code that allocates and uses unmanaged resources, like files, sockets)
    • Extended lifetime of objects
    • Incorrect understanding of how GC and .NET memory management works
    • Bugs in the .NET runtime

    The first two is usually handled by two different pieces of code:

    • Implementing IDisposable on the object and disposing of the unmanaged memory/resource in the Dispose method
    • Implementing a finalizer, to make sure unmanaged resources are deallocated when GC has found the object to be eligible for collection

    The third, however, is different.

    Say you are using a big list holding thousands of objects, totalling a significant size of memory. If you keep around a reference to this list for longer than you need to, you will have what looks like a memory leak. In addition, if you keep adding to this list, so that it grows with more data periodically, and old data is never reused, you definitely have a memory leak.

    One source of this I've seen frequently is to attach methods to event handlers but forget to unregister them when you're done, slowly bloating the event handler both in size and code to execute.

    The fourth, an incorrect understanding of how .NET memory management works can mean that you look at the memory usage in a process viewer and notice that your app keeps growing in memory usage. If you have lots and lots of memory available, GC might not run that often, giving you an incorrect picture of the current usage of memory, as opposed to mapped memory.

    The fifth, that's harder, I've only seen one resource management bug in .NET so far and afaik it has been slated for fix in .NET 4.0, it was with copying the desktop screen into a .NET image.


    Edit: In response to the question in the comments, how to avoid keeping references longer than necessary, then the only way to do that is to to just that.

    Let me explain.

    First, if you have a long-running method (for instance, it might be processing files on disk, or downloading something, or similar), and you used a reference to a big data-structure early in the method, before the long-running part, and then you don't use that data structure for the rest of the method, then .NET, in release-builds (and not running under a debugger) is smart enough to know that this reference, though it is held in a variable that is technically in scope, is eligible for garbage collection. The garbage collector is really aggressive in this respect. In debug-builds and running under a debugger, it will keep the reference for the life of the method, in case you want to inspect it when stopped at a breakpoint.

    However, if the reference is stored in a field reference in the class where the method is declared, it's not so smart, since it's impossible to determine whether it will be reused later on, or at least very very hard. If this data structure becomes unnecessary, you should clear the reference you're holding to it, so that GC will pick it up later.

    0 讨论(0)
  • 2021-02-04 10:01

    One of the many good things you can do to manage memory effectively.

    Call dispose on any object that implements IDisposable wherever possible especially on the DataSet or DataTable object.

    Better still use the using{} constructs on these objects.

    0 讨论(0)
  • 2021-02-04 10:01

    Memory leaks can come from a variety of sources.

    • You registered an event but forgot to unregister that event.

    • Files/connections were opened but not closed properly.

    • The Dispose() call was not implemented properly.

    • The Dispose() call was bypassed somehow; for example, an exception was encountered in between.

    • You encountered a deadlock (which may result in not releasing root objects).

    • The finalizer thread is blocked; for example, the STA thread for the COM object is not available.

    • There are leaks from unmanaged code.

    • An object’s lifetime is too high.

    • There is a circular reference.

    • Leaks can come from the .NET Runtime environment (on rare occasions).

    • Leaks may also appear from your testing framework. (In that case, it is a test leak, not a development environment leak, and test engineers are responsible for resolving it.)

    0 讨论(0)
  • 2021-02-04 10:03

    To answer your last two questions:

    There are, by definition, no memory leaks in managed code. There are two kinds of leaks that can occur:

    • Objects are released when no references to the object are alive. If you still have a reference to an object, it will not be released. This happens for instance when you throw away a reference to an object that has registered for an event - if you don't manually unregister the event handler (or use a weak reference), the event will still reference the object, so it will not be released although you have no obvious reference to it anymore.
    • Unmanaged resources can be leaked. Typically, a wrapper for unmanaged resources implements IDisposable and will free the resource when you call Dispose. If you just throw away the object, it will not release the resource and therefore leak it.

    So, two rules of thumb are:

    • Unregister any event handler when you release an object, or use a weak reference for that (search on SO, it's explained somewhere).
    • If a class exposes a Dispose method, call it. If you use the object only temporarily, use the using construct. If you have members that implement IDisposable, implement IDisposable yourself and call the members' Dispose in your Dispose.
    0 讨论(0)
  • 2021-02-04 10:03

    Following are the main reason of memory leak.

    1. Holding references to managed objects (classic case of event handler not released.)
    2. Failing to release unmanaged resources
    3. Failing to dispose Drawing objects
    4. I have noticed timer also causes the memory leak. Do dispose it if you are using any of the timer.

    please read further on http://blogs.msdn.com/b/davidklinems/archive/2005/11/16/three-common-causes-of-memory-leaks-in-managed-applications.aspx

    0 讨论(0)
提交回复
热议问题