剛才修改UI編輯器一個小功能,莫名其妙出現內存泄露。經過跟蹤,發現是 _variant_t 在搗鬼,來看這句代碼:
1 bool AddToCollection(IUnknown *lpiFile);
2 bool AddToCollection(VARIANT &varFile);
3 _variant_t GetDSFile();
4
5 AddToCollection(GetDSFile()); // 這句將導致內存泄露!
初看沒什么問題,但上面的代碼最后一句將的確導致內存泄露!
改成如下方式后正常:
1 bool AddToCollection(IUnknown *lpiFile);
2 bool AddToCollection(VARIANT &varFile);
3 _variant_t GetDSFile();
4
5 _variant_t varFile = GetDSFile();
6 AddToCollection(varFile); // 正常!
進一步跟蹤發現,第一種方式調用的
AddToCollection(IUnknown *lpiFile) ,檢查 _variant_t 的重載代碼如下:
1 // Extracts a VT_UNKNOWN into an IUnknown*
2 //
3 inline _variant_t::operator IUnknown*() const
4 {
5 if (V_VT(this) == VT_UNKNOWN) {
6 if (V_UNKNOWN(this) != NULL) {
7 V_UNKNOWN(this)->AddRef(); // 就是這里增加了引用計數
8 }
9 return V_UNKNOWN(this);
10 }
11
12 _variant_t varDest;
13 varDest.ChangeType(VT_UNKNOWN, this);
14
15 if (V_UNKNOWN(&varDest) != NULL) {
16 V_UNKNOWN(&varDest)->AddRef(); // 就是這里增加了引用計數
17 }
18
19 return V_UNKNOWN(&varDest);
20 }
注意到,上面的代碼中有兩處增加了引用計數!
問題就出在這里!
一般來講,從函數返回值返回的 COM 指針是不增加引用計數的,如果調用者需要持有該指針,由調用者自己增加引用計數即可,
但不知為何,這里會增加引用計數,不解!!!!!
另外
GetDSFile() 返回的是 _variant_t ,為什么編譯器要去調用
AddToCollection(IUnknown *lpiFile) ? 其實是因為
GetDSFile() 返回的是臨時變量,臨時變量優先匹配 const 的重載,所以將
bool AddToCollection(VARIANT &varFile); 改成
bool AddToCollection(const VARIANT &varFile); 也可以!!