DLL usage problem

    This site uses cookies. By continuing to browse this site, you are agreeing to our Cookie Policy.

    • DLL usage problem

      First of all, thank you Nebuchadnezzar for your help with my previous question. I got the Win32 Sys Prog 2nd book, and now have made the shared memory "volatile" and have create an event with the waitforsingleobject to control the thread. It works well.

      I am having a new problem now. I want to create a DLL that has several common functions that DX programs will use. I added a function to it to load an Xfile, extract the textures and materials, add normals to the mesh, and optimize it.

      My game runs fine until I exit. Then during the cleanup phase it bombs with an exception when it tries to delete the first class object that contains the first mesh. I traced it through the destructors. But it bombs in the dbgheap.c:

      Source Code

      1. #else /* WINHEAP */
      3. /*
      4. * Go through the heap regions and see if the pointer lies within one
      5. * of the regions of the local heap.
      6. *
      7. * Pointers from non-local heaps cannot be handled. For example, a
      8. * non-local pointer may come from a DLL that has the CRT linked-in.
      9. *
      10. */
      12. for (i = 0; (base = _heap_regions[i]._regbase) != NULL &&
      13. i < _HEAP_REGIONMAX; i++)
      14. {
      15. if (pUserData >= base && pUserData <
      16. (void *)(((char *)base)+_heap_regions[i]._currsize))
      17. return TRUE;
      18. }
      20. return FALSE;
      22. #endif /* WINHEAP */
      23. --> } (THE GREEN POINTER IS HERE)
      Display All


      If I put the exact same code in a function, instead of the DLL, I get no errors. So, it is only with the DLL function call that I get the problem.

      What would be unique with the DLL memory management that could cause this? It seems to be a heap issue. I tried allocating memory for some strings with "new" instead of just as static in size, but no difference.

      Oh, and I don't get any problems with the small test program I wrote to test/debug the DLL code. I only get it when I use the DLL in my game.

      The second thread is not started until after the DLL has been used to create the meshes for the game.

      Thanks.

    • RE: DLL usage problem

      I have attempted to trace it through every instruction being executed on the exit. It goes to the destructor of the derived class and executes the release of the textures, but there are no textures for that mesh, so it really does nothing. Then it hits the end of that destructor and I have a breakpoint at the beginning of the destructor in the base class, but it doesn't get there. It gets that error and stops there.

      I am baffled. :baby:

      I have been reading all the books I have on memory management. No clue.

      The post was edited 1 time, last by GamerRick ().

    • Does it do the same thing with a release build? You may be able to use an assert to catch the error - in other words, assert(test condition); prior to the call that causes the crash. Something odd might be happening.

      Addionally, the problem might be that the memory being cleared isn't "local" to the dll. You might have to leave the cleanup to the main program instead of using common functions in the dll to do it. I don't usually fiddle much with dll's, so I'm not more than 50% sure of that....

      Rich
      "Your job is not to die for your country. Your job is to make some other poor sod die for his."

    • The DLL is only used during initialization to load the xfiles. I have a class (called XFileObject) that contains the resulting mesh and all associated stuff (materials, textures, etc). So, I don't know why the DLL would cause the problem at exit time.

      Source Code

      1. #define EXPORT __declspec (dllexport)
      3. EXPORT HRESULT CALLBACK DXLoadXfile(IDirect3DDevice9* pDevice,
      4. LPWSTR Xfn,
      5. std::vector<D3DMATERIAL9>& XFMtrls,
      6. std::vector<IDirect3DTexture9*>& XFTextures,
      7. ID3DXMesh** ppObjectMesh,
      8. UINT OPTIONS)
      9. {
      10. HRESULT hr;
      11. ID3DXMesh* ObjectMesh = NULL;
      12. const UINT TpathL = 259;
      13. const UINT UFnL = 100;
      14. TCHAR Tpath[260];
      15. TCHAR UFn[200];
      16. TCHAR CurrPath[260];
      18. DWORD CPL = GetCurrentDirectory(259, CurrPath); // save current directory
      19. if (CPL == NULL)
      20. return E_FAIL;
      22. size_t FnLength;
      24. ID3DXBuffer* adjBuffer = 0; // buffer for adjacency info
      25. ID3DXBuffer* mtrlBuffer = 0; // buffer for materials
      26. DWORD numMtrls = 0;
      28. hr = D3DXLoadMeshFromX(
      29. Xfn, // Xfile to load
      30. D3DXMESH_MANAGED,
      31. pDevice,
      32. &adjBuffer, // Pointer to adjacency buffer
      33. &mtrlBuffer, // Pointer to material buffer
      34. 0, // effect instances (unused)
      35. &numMtrls, // Number of materials
      36. &ObjectMesh); // ID3DXMesh
      38. if (FAILED(hr))
      39. return hr;
      41. if ((OPTIONS & OPT_TEXSAMEASX) != 0) // Use path from xfile
      42. {
      43. hr = StringCchLength(Xfn, TpathL, &FnLength);
      44. if (FAILED(hr))
      45. return hr;
      47. // Save path of xfile to use for textures
      48. hr = StringCchCopy(Tpath, FnLength + 1, Xfn);
      49. if (FAILED(hr))
      50. return hr;
      52. size_t pos;
      53. for ( pos = FnLength; pos > 0; pos-- )
      54. {
      55. if (Tpath[pos] == '\\')
      56. break;
      57. }
      59. if (pos > 0)
      60. Tpath[pos+1] = NULL;
      61. else
      62. Tpath[0] = NULL; // there is no path
      64. SetCurrentDirectory(Tpath);
      65. }
      67. ID3DXBuffer* errors = 0;
      68. DWORD* Tadj = static_cast<DWORD*>(adjBuffer->GetBufferPointer());
      70. hr = D3DXValidMesh(ObjectMesh, (DWORD*) adjBuffer->GetBufferPointer(), &errors);
      71. SAFE_RELEASE(errors);
      72. ID3DXMesh* TMesh = NULL;
      74. if (FAILED(hr))
      75. {
      76. try
      77. {
      78. if (hr == D3DXERR_INVALIDMESH)
      79. {
      80. hr = D3DXCleanMesh(D3DXCLEAN_SIMPLIFICATION,
      81. ObjectMesh, Tadj, &TMesh,
      82. Tadj,
      83. &errors);
      85. if( errors )
      86. {
      87. MessageBoxA(0, (char*)errors->GetBufferPointer(), 0, 0);
      88. throw hr;
      89. }
      90. if (FAILED(hr))
      91. {
      92. MessageBox(0, L"D3DXCleanMesh - Failed", L"ERROR", 0);
      93. throw hr;
      94. }
      96. ObjectMesh->Release();
      97. ObjectMesh = TMesh;
      98. TMesh = NULL;
      100. }
      101. else
      102. throw hr;
      103. }
      104. catch (HRESULT hr)
      105. {
      106. SAFE_RELEASE(errors);
      107. SAFE_RELEASE(adjBuffer);
      108. SAFE_RELEASE(ObjectMesh);
      109. SAFE_RELEASE(TMesh);
      110. return hr;
      111. }
      112. }
      114. try
      115. {
      116. if( mtrlBuffer != 0 && numMtrls != 0 )
      117. {
      118. D3DXMATERIAL* mtrls = (D3DXMATERIAL*)mtrlBuffer->GetBufferPointer();
      120. for(DWORD i = 0; i < numMtrls; i++)
      121. {
      122. // the MatD3D property doesn't have an ambient value set
      123. // when its loaded, so set it now:
      124. mtrls[i].MatD3D.Ambient = mtrls[i].MatD3D.Diffuse;
      126. // save the ith material
      128. XFMtrls.push_back( mtrls[i].MatD3D );
      130. // check if the ith material has an associative texture
      132. if( mtrls[i].pTextureFilename != NULL )
      133. {
      134. size_t cc;
      135. hr = StringCchLengthA(mtrls[i].pTextureFilename, UFnL, &cc);
      136. if (FAILED(hr))
      137. throw E_FAIL;
      139. //size_t cc = mbstowcs(UFn, mtrls[i].pTextureFilename, 99); // convert char* string to UNICODE
      140. int MBL = MultiByteToWideChar(CP_ACP, NULL, mtrls[i].pTextureFilename, int(cc+1),
      141. UFn, UFnL);
      142. if (MBL <= 0 || MBL > UFnL)
      143. throw E_FAIL;
      145. if ((OPTIONS & OPT_TEXSAMEASX) != 0) // Use path from xfile
      146. {
      147. size_t pos = MBL;
      148. TCHAR* pUFn = UFn+MBL; // point to end of file name
      149. while ( pos > 0 && *pUFn != TCHAR('\\'))
      150. {
      151. pos--;
      152. pUFn--;
      153. }
      155. hr = StringCchCopy(Tpath, TpathL, pUFn);
      156. if (FAILED(hr))
      157. throw hr;
      158. }
      159. else // use path and file name from Xfile
      160. {
      161. hr = StringCchCopy(Tpath, TpathL, UFn);
      162. if (FAILED(hr))
      163. throw hr;
      164. }
      166. IDirect3DTexture9* tex = 0;
      167. hr = D3DXCreateTextureFromFile(pDevice,
      168. Tpath,
      169. &tex);
      171. if (FAILED(hr))
      172. throw hr;
      174. // save the loaded texture
      175. XFTextures.push_back( tex );
      176. }
      177. else
      178. {
      179. // no texture for the ith subset
      180. XFTextures.push_back( NULL );
      181. }
      182. } // end for
      183. } // End if materials
      184. }
      185. catch (HRESULT hr)
      186. {
      187. SAFE_RELEASE(mtrlBuffer);
      188. SAFE_RELEASE(adjBuffer);
      189. SAFE_RELEASE(ObjectMesh);
      190. for(UINT i = 0; i < XFTextures.size(); i++)
      191. SAFE_RELEASE(XFTextures[i]);
      192. XFTextures.clear();
      193. XFMtrls.clear();
      194. return hr;
      195. }
      197. SAFE_RELEASE(mtrlBuffer); // done w/ buffer
      199. if ((OPTIONS & OPT_GENNORMALS) != 0)
      200. {
      201. D3DVERTEXELEMENT9 elems[MAX_FVF_DECL_SIZE];
      202. bool hasNormals = false;
      203. hr = ObjectMesh->GetDeclaration(elems);
      204. if (SUCCEEDED(hr))
      205. {
      206. for(int i = 0; i < MAX_FVF_DECL_SIZE; ++i)
      207. {
      208. // Did we reach D3DDECL_END() {0xFF,0,D3DDECLTYPE_UNUSED, 0,0,0}?
      209. if(elems[i].Stream == 0xff)
      210. break;
      212. if( elems[i].Type == D3DDECLTYPE_FLOAT3 &&
      213. elems[i].Usage == D3DDECLUSAGE_NORMAL &&
      214. elems[i].UsageIndex == 0 )
      215. {
      216. hasNormals = true;
      217. break;
      218. }
      219. }
      220. }
      221. else
      222. {
      223. DWORD meshFVF = ObjectMesh->GetFVF();
      224. if ((meshFVF & D3DFVF_NORMAL) == 0)
      225. hasNormals = false;
      226. }
      228. ID3DXMesh* tempMesh = 0;
      230. try
      231. {
      232. if (!hasNormals)
      233. {
      234. hr = ObjectMesh->CloneMeshFVF(D3DXMESH_SYSTEMMEM, D3DFVF_XYZ | D3DFVF_NORMAL | D3DFVF_TEX2 ,
      235. pDevice, &tempMesh);
      236. if (FAILED(hr))
      237. throw hr;
      239. hr = D3DXComputeNormals(tempMesh, 0);
      240. if (FAILED(hr))
      241. throw hr;
      243. SAFE_RELEASE(ObjectMesh);
      245. ObjectMesh = tempMesh;
      246. tempMesh = NULL;
      248. hr = ObjectMesh->GenerateAdjacency(0.001f, Tadj);
      249. if (SUCCEEDED(hr))
      250. {
      251. hr = ObjectMesh->OptimizeInplace(
      252. D3DXMESHOPT_ATTRSORT |
      253. D3DXMESHOPT_COMPACT |
      254. D3DXMESHOPT_VERTEXCACHE,
      255. Tadj,
      256. 0, 0, 0);
      257. }
      258. if (FAILED(hr))
      259. throw hr;
      260. }
      261. else
      262. {
      263. hr = ObjectMesh->OptimizeInplace(
      264. D3DXMESHOPT_ATTRSORT |
      265. D3DXMESHOPT_COMPACT |
      266. D3DXMESHOPT_VERTEXCACHE,
      267. Tadj,
      268. 0, 0, 0);
      269. if (FAILED(hr))
      270. throw hr;
      271. }
      272. }
      273. catch (HRESULT hr)
      274. {
      275. SAFE_RELEASE(adjBuffer);
      276. SAFE_RELEASE(ObjectMesh);
      277. SAFE_RELEASE(tempMesh);
      278. for(UINT i = 0; i < XFTextures.size(); i++)
      279. SAFE_RELEASE(XFTextures[i]);
      280. XFTextures.clear();
      281. XFMtrls.clear();
      282. return hr;
      283. }
      284. }
      286. SAFE_RELEASE(adjBuffer);
      288. *ppObjectMesh = ObjectMesh;
      289. SetCurrentDirectory(CurrPath);
      290. return hr;
      291. }
      Display All