FreeTypeFont not freeing d_fontData correctly

[Ident]

Original report by Anonymous.

---

Automatic migration. Original reporter: "nightwolf"

In file CEGUIFreeTypeFont.cpp, line 375, the class member d_fontData is freed by calling the Release() method, which is not correct and produces errors when using a custom memory manager.

Please free the d_fontData member using unloadRawDataContainer() from the current resource provider, allowing custom resource providers to free the container themselves.

If the custom memory manager pads the memory buffers, thus returning slightly different pointers from the ones returned by the CRT, this bug causes an exception to be raised in debug mode by the runtime library itself.

Reproducibility: always

OS: Windows, OS build: XP, Platform: PC

[Ident]

Original comment by Anonymous.

---

Original reporter: zap

Here is a patch file that fixes two things:

• Calls unloadRawDataContainer() as suggested above
• Fixes for infinite looping in some special cases in FreeTypeFont.cpp

[Ident]

Original comment by Anonymous.

---

Original reporter: spannerman

This patch has been tested successfully as described here:
http://www.cegui.org.uk/phpBB2/viewtopic.php?t=1687

[Ident]

Original comment by Anonymous.

---

Original reporter: lindquist

Patch has been applied to svn :)