Problem with eglSwapBuffers and heap corruption

I’ve got a bit of a problem with the use of eglSwapBuffers in my Series 60 code.

I was running my app full screen which seemed to automatically make it use the direct screen access code.

When I displayed the menu I was hitting memory corruption errors. I tracked this down to the menu display causing loss of direct screen access, which freed the bitmap being used for this but OpenGL still had a pointer to this now dead bitmap and in the call to glClear it corrupted the heap trying to write to this dead pointer.

I debugged into the sample apps to see how they avoided this problem and discovered that apps like Example3D and GlExample were calling eglSwapBuffers at the start of their draw cycles and this call sorted out the dead pointer problem and stopped the heap corruption.

This solution also fixed my problem but it doesn’t seem to be right thing to do, surely eglSwapBuffers should be called straight after the drawing code to copy it to the display ASAP. The method used in those sample apps is postponing the screen update until the next animation cycle causing an unwanted lag in updating the display.

I’ve just had a look through some of the other sample apps and believe they are surviving this problem by luck, the ones that run in full screen but don’t call eglSwapBuffers until after their draw cycle are actually drawing to freed memory but getting away with it because writing to freed heap space doesn’t actually corrupt anything unless some other code tries to use that heap.

I’ve confirmed this by making the following change in the \OpenGlExamples\3Dexamples\openglex\rain sample.

TInt CRainContainer: :DrawCallBack( TAny* aInstance )
{
CRainContainer* instance = (CRainContainer*) aInstance;
if ( instance->iRain->GetState() == CRain::ERunning )
{
instance->iFrame++;
/* Call the main OpenGL ES Symbian rendering ‘loop’ */
// Added following alloc and heap checks to churn and test the heap
User::Heap().Check();
TAny *xxx1=User::Alloc(100);
User::Heap().Check();
instance->iRain->AppCycle( instance->iFrame );
User::Heap().Check();
User::Free(xxx1);
User::Heap().Check();

In the first call to User::Heap().Check(); after the menu was displayed the code paniced with User-49, what has happened is the alloc of 100 bytes has gone into the space freed by deleting the direct screen buffer, the opengl draw code has then trashed the free cell pointer at the end of this new alloc cell, which was picked up by the heap check afterwards.

At the moment I can’t see any solution other than calling eglSwapBuffers before my draw code, and having to put up with the lag in updating the display.

But I suggest this whole situation needs to be looked at carefully to prevent other people hitting this same problem I did with heap corruption, it was very very hard to track down, involved writing a heap dump function to analyse what was corrupting my heap, lots of memory watch break points to track back step by step who was responsible for the corruption, then watching the same memory watch points in the sample apps to work out why they didn’t hit the same problem as me.

At the very least all the sample apps need to be changed so they aren’t writing to freed memory.

Steve Townsend
Great Ape Software Ltd
http://www.GreatApe.com/cv

Hi.

What implementation are you using? On-device implementation (in 6630, 6680, 6681, …) or an external implementation such as Hybrid Gerbera?

If you are using on-device Nokia implementation running on a Nokia phone, please forward this to Forum Nokia.

If you are running on Gerbera, then you should forward this to Hybrid Gerbera support forums.

–jani;

It’s the version in Series 60 2nd edition SDK running under WINS, I’ll post it on forum Nokia, thanks.

This topic was automatically closed 183 days after the last reply. New replies are no longer allowed.