#include "terrain.h"
bool CTERRAIN::LoadHeightMap( char* szFilename, int iSize )
{
 FILE* pFile;
 if( m_heightData.m_pucData ) UnloadHeightMap( );
 m_heightData.m_pucData= new unsigned char [iSize*iSize];
 if( m_heightData.m_pucData==NULL )
 {
  printf( "Could not allocate memory for%s\n", szFilename );
  return false;
 }

 fread( m_heightData.m_pucData, 1, iSize*iSize, pFile );
 fclose( pFile );

 m_heightData.m_iSize = iSize;
 m_iSize = m_heightData.m_iSize;

 printf( "Loaded %s\n", szFilename );
 return true;
}


bool CTERRAIN::UnloadHeightMap( void )
{
 if( m_heightData.m_pucData )
 {
  delete[] m_heightData.m_pucData;
  m_heightData.m_iSize= 0;
 }
 printf( "Successfully unloaded the height map\n" );
 return true;
}

Quote:Original post by swiftcoder
There is nothing really wrong with that code, except that it doesn't check whether the right amount of data was actually read - try replacing this:

 fread( m_heightData.m_pucData, 1, iSize*iSize, pFile ); 

with this:
*** Source Snippet Removed ***

Quote:Original post by kauna
Hi,

so does the fault actually occure inside this code block ?

You could run your program with a debugger and see which line causes the error.

Regards

[edit]

I hope that you set m_heightData.m_pucData to NULL before trying to load

pFile = fopen(szFilename, "rb");

Quote:Original post by Solias
Looks like you read from the uninitialized pFile variable without opening the file.

add something like:

pFile = fopen(szFilename, "rb");

Be sure to check the return value in case the file doesn't exist or you don't have permission to open it. (Note that "rb" is the windows version, linux might be different).