我已经追踪到了一个泄漏,但我自己似乎无法理解(修复)它。我首先使用 ANTS 内存分析器来确保我的代码实际上是堆栈内存。它一开始使用 25 MB,但在一个小时左右的时间内就使用了超过 100 MB。我正在为我的一个 friend 编写此代码,实际上他一直在使用这个有缺陷的程序,他用完了整个 18 GB 的内存,并出现了内存不足的异常。
泄漏部分对于程序来说并不重要,但如果没有 RefreshSessions() 方法,它几乎毫无用处。
我一直在扩展项目Vista Core Audio API Master Volume Control来自代码项目。
这是似乎泄漏的部分。经过不使用测试,不漏水。
更新:
public void RefreshSessions()
{
Marshal.ThrowExceptionForHR(_AudioSessionManager.GetSessionEnumerator(out _SessionEnum));
_Sessions.Refresh(_SessionEnum);
}
(从此处删除了类代码)
我编写的代码不多,所以我可能错过了一些东西,但如果需要更多详细信息,您可以实际下载源代码,或者我可以尽我所能回答。
(此处删除了不必要的代码)
使用这个简单的控制台应用程序测试了泄漏:
namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
MMDeviceEnumerator DevEnum = new MMDeviceEnumerator();
MMDevice device = DevEnum.GetDefaultAudioEndpoint(EDataFlow.eRender, ERole.eMultimedia);
Console.ReadKey();
int i = 0;
while (i < 10000)
{
device.AudioSessionManager.RefreshSessions();
i++;
}
Console.ReadKey();
}
}
}
更新2
我想我已经解决了。必须运行一些更长的测试,但至少内存使用情况似乎已经稳定。这个想法来自 dialer,他在 C++ 中找到了漏洞的修复方法。
public void RefreshSessions()
{
_Sessions.Release(); //added this
IAudioSessionEnumerator _SessionEnum;
Marshal.ThrowExceptionForHR(_AudioSessionManager.GetSessionEnumerator(out _SessionEnum));
_Sessions.Refresh(_SessionEnum);
}
这是SessionCollection
中的部分:
public void Release()
{
Marshal.ReleaseComObject(_AudioSessionEnumerator);
}
这并不完全是建议的代码拨号器(我最终还是使用了它),但仍然如此。 正如他所说,这可能不是实现这一目标的最佳方法,但我会选择它,因为它似乎不会对我的应用程序产生任何不利影响。
最佳答案
另一次编辑
public void RefreshSessions()
{
if (_SessionEnum != null)
{
Marshal.ReleaseComObject(_SessionEnum);
}
Marshal.ThrowExceptionForHR(_AudioSessionManager.GetSessionEnumerator(out _SessionEnum));
}
以上代码显式释放了 SessionEnum,并修复了 C# 中的泄漏。 不过,这可能应该以更好的方式处理。
编辑:
以下 C++ 程序与您在循环测试程序中所做的等效。 for 循环末尾的 Release
调用修复了泄漏。我今天需要走了,也许你可以玩一下并尝试自己修复它。或者也许其他人可以找出并解释为什么 CLR 垃圾收集器不会在上面的 C# 程序中的某个时刻自动调用 Release
。
#include <stdio.h>
#include <tchar.h>
#include <audiopolicy.h>
#include <mmdeviceapi.h>
#define SAFE_RELEASE(p) { if ( (p) ) { (p)->Release(); (p) = 0; } }
#define CHECK_HR(hr) if (FAILED(hr)) { goto done; }
const CLSID CLSID_MMDeviceEnumerator = __uuidof(MMDeviceEnumerator);
const IID IID_IMMDeviceEnumerator = __uuidof(IMMDeviceEnumerator);
const IID IID_IAudioSessionManager2 = __uuidof(IAudioSessionManager2);
int _tmain(int argc, _TCHAR* argv[])
{
HRESULT hr;
CoInitialize(0);
IMMDeviceEnumerator *deviceEnum = 0;
CHECK_HR(hr = CoCreateInstance(
CLSID_MMDeviceEnumerator, NULL,
CLSCTX_ALL, IID_IMMDeviceEnumerator,
(void**)&deviceEnum));;
IMMDevice *endpoint = 0;
CHECK_HR(deviceEnum->GetDefaultAudioEndpoint(eRender, eMultimedia, &endpoint));
getchar();
// lazy initialization as found in MMDevice.AudioSessionManager..get
IAudioSessionManager2 *m = 0;
CHECK_HR(endpoint->Activate(IID_IAudioSessionManager2, CLSCTX_ALL, 0, (void **)&m));
for (int i = 0; i < 100000; i++)
{
IAudioSessionEnumerator *sessEnum = 0;
m->GetSessionEnumerator(&sessEnum);
sessEnum->Release(); // leak
}
getchar();
printf("success");
return 0;
done:
printf("failure");
return 1;
}
旧
我的猜测:
_AudioSessionManager.GetSessionEnumerator(out _SessionEnum)
生成一个枚举器。当您调用构造函数 SessionCollection(_SessionEnum)
时,将枚举 _SessionEnum
。每个枚举步骤都会检索一个实际的非托管对象。
如果它是值类型,那么它实际上会被复制到 session 集合中(请记住 List(IEnumerable e)
构造函数复制每个元素)。然后,副本将被垃圾收集,但原始对象是从非托管代码分配的,并会产生泄漏。如果是这种情况,您应该在调用 Collection 构造函数后立即使用一些非管理内存释放函数释放内存。
如果它是引用类型,它也不会被释放,因为内存中的实际对象不会被垃圾回收,因为它是从非托管代码中分配的。如果是这种情况,当您不再需要具有非托管库函数的对象时,您需要释放它们的内存。
关于c# - 非托管代码中的内存泄漏?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/15692126/