CE5.0数据库问题:CeReadRecordProps()的使用与内存泄露

rong2046

CE5.0数据库问题:CeReadRecordProps()的使用与内存泄露 [复制链接]

偶对数据库操作函数不是很熟,今天看到一同事写的CeReadRecordProps()函数的使用.怀疑它产生了内存泄露,请大家帮忙分析一下,代码如下:
PBYTE pBuff;

for(j = 0; j < iRecordCount; j++)
{
  //移动记录指针
  ceOid = CeSeekDatabase(m_hDB,CEDB_SEEK_BEGINNING,j,&dwIndex);
  if(ceOid == 0)
  {
    AfxMessageBox(_T("读记录失败!!"));
    return(FALSE);
  }
  
  pBuff = 0;
  ceOid = CeReadRecordProps(m_hDB,CEDB_ALLOWREALLOC,&wProps,NULL,&(LPBYTE)pBuff,&dwRecSize);
  if(ceOid == 0)
  {
    AfxMessageBox(_T("读记录失败!!!"));
    return(FALSE);
  }
  pRecord = (PCEPROPVAL)pBuff;
  for (i = 0; i < wProps; i++)
  {
    // 数据分析
    pRecord++;
  }
}

CeReadRecordProps()函数的参数:pBuff,与CEDB_ALLOWREALLOC的使用:CeReadRecordProps()在使用了CeReadRecordProps参数时,将调用LocalAlloc()为pBuff分配内存.
现在的问题是:分配到的内存pBuff,在何时释放?及如何释放?

在网上看,大家在讨论是使用free,free[],还是LocalFree()来释放.

特别的,在代码中,强制给pBuff赋值为零的代码,使的API有可能再也找不到以前分配的内存,所以导致内存泄露.这是我的观点,大家帮忙看看,多谢!

juliasarah

当然,也有说不用释放的.因为代码中没有出现new/malloc等分配内存的语句.

zzxxzz

用 LocalFree 来释放啊

kangxin

1. 
   
2. pBuff = 0;
   
3.   ceOid = CeReadRecordProps(m_hDB,CEDB_ALLOWREALLOC,&wProps,NULL,&(LPBYTE)pBuff,&dwRecSize);
   

复制代码

&(LPBYTE)pBuff 0x00000000上的数据是未知的, 写数据到这个数据为地址的空间, 当然就会crash了..

核桃佳子

引用 3 楼 flandy1982 的回复:
C/C++ code
pBuff=0;
  ceOid= CeReadRecordProps(m_hDB,CEDB_ALLOWREALLOC,&wProps,NULL,&(LPBYTE)pBuff,&dwRecSize);
&(LPBYTE)pBuff 0x00000000上的数据是未知的, 写数据到这个数据为地址的空间, 当然就会crash了..

哈哈...,看来你也不懂,去看看帮助吧,说的很清楚的
如果pBuff为空,API会自己分配内存.否则就没有这个问题了.

API分配内存是可以肯定的,但是什么问题释放分配的内存不清楚!

nsz

貌似ＣＥ中有检查内存泄露的专用工具吧，是否测试过，能得知泄露了么？

呵~~~

以前整过别人写的绘图的东东(pc下的程序)，貌似CＡＤ的一个软件，他明明动态申请了内存，但没有释放，程序未泄露，而当我加上delete后，在界面上绘图反而出现了莫名的错误，比如画个曲线，结果弧度显示不出来

gorey

如果pBuff为空,API会自己分配内存.否则就没有这个问题了.


有道理！！

fanghuyu

引用 4 楼 91program 的回复:
引用 3 楼 flandy1982 的回复:
C/C++ code
pBuff=0;
? ceOid= CeReadRecordProps(m_hDB,CEDB_ALLOWREALLOC,&wProps,NULL,&(LPBYTE)pBuff,&dwRecSize);
&(LPBYTE)pBuff 0x00000000上的数据是未知的, 写数据到这个数据为地址的空间, 当然就会crash了..


哈哈...,看来你也不懂,去看看帮助吧,说的很清楚的
如果pBuff为空,API会自己分配内存.否则就没有这个问题了.

API分配内存是可以肯定的,但是什么问题释放分配的内存不清楚!

原来如此
微软的代码是没有释放内存的...
C:\WINCE500\PUBLIC\COMMON\OAK\DRIVERS\BLUETOOTH\SAMPLE\BTDIALP\btdialp.cxx

happyjinwen

不像是这个文件中的代码

a2479104

看这个函数

1. 
   
2. void CePimCommand(HWND hWnd, PIMTYPE ptData, UINT uDataCount, HANDLE *rghData, void *pReserved) {
   
3.         if (uDataCount != 1)
   
4.                 return;
   
5. 
   
6.         CEOID oid = *(CEOID *)rghData;
   
7. 
   
8.         CEOID db_out = 0;
   
9. 
   
10.         HANDLE hdb = CeOpenDatabase (&db_out, L"Contacts Database", 0, 0, NULL);
    
11.         if (hdb == INVALID_HANDLE_VALUE)
    
12.                 return;
    
13. 
    
14.         DWORD dwIndex = 0;
    
15. 
    
16.         if (! CeSeekDatabase (hdb, CEDB_SEEK_CEOID, (DWORD)oid, &dwIndex))
    
17.                 return;
    
18. 
    
19.         unsigned char *lpBuff = NULL;
    
20.         DWORD cBuff = 0;
    
21. 
    
22.         CEPROPID aPropID[PHONE_NUM] = {
    
23.                 PR_HOME_TELEPHONE_NUMBER_W,
    
24.                 PR_HOME2_TELEPHONE_NUMBER_W,
    
25.                 PR_BUSINESS_TELEPHONE_NUMBER_W,
    
26.                 PR_CALLBACK_TELEPHONE_NUMBER_W,
    
27.                 PR_CAR_TELEPHONE_NUMBER_W,
    
28.                 PR_MOBILE_TELEPHONE_NUMBER_W,
    
29.                 PR_RADIO_TELEPHONE_NUMBER_W,
    
30.                 PR_PRIMARY_TELEPHONE_NUMBER_W };
    
31. 
    
32.         WORD dwPropIDs = PHONE_NUM;
    
33. 
    
34.         [color=#FF0000]if (! CeReadRecordProps (hdb, CEDB_ALLOWREALLOC, &dwPropIDs, aPropID, &lpBuff, &cBuff))
    
35.                 return;[/color]
    
36.         PhoneInfo numbers[PHONE_NUM];
    
37. 
    
38.         memset (numbers, 0, sizeof(numbers));
    
39.         CEPROPVAL *ppv = (CEPROPVAL *)lpBuff;
    
40. 
    
41.         int iPhoneCnt = 0;
    
42. 
    
43.         for (int i = 0 ; (i < dwPropIDs) && (iPhoneCnt < PHONE_NUM); ++i, ++ppv) {
    
44.                 if (ppv->wFlags & CEDB_PROPNOTFOUND)
    
45.                         continue;
    
46. 
    
47.                 switch (ppv->propid) {
    
48.                 case PR_HOME_TELEPHONE_NUMBER_W:
    
49.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"h");
    
50.                         break;
    
51.                 case PR_HOME2_TELEPHONE_NUMBER_W:
    
52.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"h2");
    
53.                         break;
    
54.                 case PR_BUSINESS_TELEPHONE_NUMBER_W:
    
55.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"w");
    
56.                         break;
    
57.                 case PR_CALLBACK_TELEPHONE_NUMBER_W:
    
58.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"c/back");
    
59.                         break;
    
60.                 case PR_CAR_TELEPHONE_NUMBER_W:
    
61.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"car");
    
62.                         break;
    
63.                 case PR_MOBILE_TELEPHONE_NUMBER_W:
    
64.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"m");
    
65.                         break;
    
66.                 case PR_RADIO_TELEPHONE_NUMBER_W:
    
67.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"radio");
    
68.                         break;
    
69.                 case PR_PRIMARY_TELEPHONE_NUMBER_W:
    
70.                         wcscpy (numbers[iPhoneCnt].szPhoneType, L"prim.");
    
71.                         break;
    
72.                 default:
    
73.                         continue;
    
74.                 }
    
75. 
    
76.                 if (wcslen (ppv->val.lpwstr) >= MAX_PHONE)
    
77.                         continue;
    
78. 
    
79.                 wcscpy (numbers[iPhoneCnt].szPhone, ppv->val.lpwstr);
    
80.                 ++iPhoneCnt;
    
81.         }
    
82. 
    
83.         if (iPhoneCnt < PHONE_NUM) {
    
84.                 numbers[iPhoneCnt].szPhone[0] = '\0';
    
85.                 numbers[iPhoneCnt].szPhoneType[0] = '\0';
    
86.         }
    
87. 
    
88.         int ndx = DialogBoxParam (ghInst, MAKEINTRESOURCE (IDD_DIALOG1), NULL, WhichDlg, (LPARAM)numbers);
    
89. 
    
90.         if ((ndx >= 0) && (ndx < iPhoneCnt)) {
    
91.                 PROCESS_INFORMATION pi;
    
92.                 STARTUPINFO                si;
    
93. 
    
94.                 memset (&pi, 0, sizeof(pi));
    
95.                 memset (&si, 0, sizeof(si));
    
96.                 si.cb = sizeof(si);
    
97. 
    
98.                 DWORD bCP = CreateProcess (L"btdialer.exe", numbers[ndx].szPhone, NULL, NULL, FALSE, 0, NULL,
    
99.                                         NULL, &si, &pi);
    
100. 
     
101.                 if (bCP) {
     
102.                         CloseHandle (pi.hThread);
     
103.                         CloseHandle (pi.hProcess);
     
104.                 }
     
105.         }
     
106. 
     
107.         return;
     
108. }
     

复制代码