My program is crashing due to a exception access violation. It happens consistently when the cache gets full and starts shifting. Whats the issue with this code?
My program is crashing due to a exception access violation. It happens consistently when the cache gets full and starts shifting. Whats the issue with this code?
#include "TopologyOwnersManagerCache.h"
#include <iostream>
using namespace std;
TopologyOwnersManagerCache::TopologyOwnersManagerCache()
{
m_cacheSize = 10;
m_cacheEmptyIndex = 0;
m_ownersManagers = new A3DTopoItemOwnersManager *[m_cacheSize];
m_brepItems = new A3DRiRepresentationItem *[m_cacheSize];
for (int i = 0; i < m_cacheSize; i++)
{
m_ownersManagers[i] = (A3DTopoItemOwnersManager *)0;
m_brepItems[i] = (A3DRiRepresentationItem *)0;
}
}
TopologyOwnersManagerCache::~TopologyOwnersManagerCache()
{
for (int i=0; i < m_cacheSize; i++)
{
if (m_ownersManagers[i])
{
// Release the map
A3DTopoItemOwnersManagerGet((A3DRiRepresentationItem *)0, m_ownersManagers[i]);
}
}
delete[] m_ownersManagers;
delete[] m_brepItems;
}
A3DTopoItemOwnersManager *
TopologyOwnersManagerCache::getOwnersManager(A3DRiRepresentationItem *brepOwner)
{
// Check if it is in the cache
for (int i=0; i < m_cacheSize; i++)
{
if (m_brepItems[i] == brepOwner)
{
return m_ownersManagers[i];
}
}
// If the cache is full, remove the first entry and shift the rest down
int cacheIndex = m_cacheEmptyIndex;
if (cacheIndex < 0)
{
cout << "Shifting cache down!" << endl;
cacheIndex = m_cacheSize - 1;
// Release the map of the first entry
cout << "About to release : " << m_ownersManagers[0] << endl;
A3DTopoItemOwnersManagerGet((A3DRiRepresentationItem *)0, m_ownersManagers[0]);
for (int i=1; i < m_cacheSize; i++)
{
m_ownersManagers[i-1] = m_ownersManagers[i];
m_brepItems[i-1] = m_brepItems[i];
}
}
m_brepItems[cacheIndex] = brepOwner;
//crash is happening here on A3DTopoItemOwnersManagerGet() call
if (A3DTopoItemOwnersManagerGet(brepOwner, m_ownersManagers[cacheIndex]) != A3D_SUCCESS)
{
return (A3DTopoItemOwnersManager *)0;
}
// Check if the cache is full now
m_cacheEmptyIndex++;
if (m_cacheEmptyIndex >= m_cacheSize)
{
m_cacheEmptyIndex = -1;
}
return m_ownersManagers[cacheIndex];
}
Try set last variable of m_ownersManagers
as 0 after shifting the cache down.
Use in end of block
if (cacheIndex < 0)
{
after loop this line:
m_ownersManagers[cacheIndex] = (A3DTopoItemOwnersManager *)0;