[Scummvm-git-logs] scummvm branch-2-0 -> 58cbd45ef5b6776b761fc3d51f531a8d12946c24
csnover
csnover at users.noreply.github.com
Sun Dec 3 19:34:59 CET 2017
This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .
Summary:
58cbd45ef5 SDL: Fix memory corruption when switching to/from 32-bit cursors
Commit: 58cbd45ef5b6776b761fc3d51f531a8d12946c24
https://github.com/scummvm/scummvm/commit/58cbd45ef5b6776b761fc3d51f531a8d12946c24
Author: Colin Snover (github.com at zetafleet.com)
Date: 2017-12-03T12:34:35-06:00
Commit Message:
SDL: Fix memory corruption when switching to/from 32-bit cursors
When a 32-bit cursor has the same size as an 8- or 16-bit cursor,
the mouse surfaces were not being regenerated even though the
32-bit cursors have a different memory requirement. This lead to
memory corruption as an inappropriate surface would be used for
the other type of cursor.
The shoe-horned 32-bit cursor support is clearly showing its
scrappy nature here and probably ought to be revisited in the
future if the SurfaceSdl graphics manager sticks around.
Fixes Trac#10349, Trac#10350, Trac#10351.
(cherry picked from commit 20b2c1b7e156f0586799f7df9d6e93c757dabeac)
Changed paths:
backends/graphics/surfacesdl/surfacesdl-graphics.cpp
diff --git a/backends/graphics/surfacesdl/surfacesdl-graphics.cpp b/backends/graphics/surfacesdl/surfacesdl-graphics.cpp
index c2e0fd2..51421c5 100644
--- a/backends/graphics/surfacesdl/surfacesdl-graphics.cpp
+++ b/backends/graphics/surfacesdl/surfacesdl-graphics.cpp
@@ -1834,12 +1834,20 @@ void SurfaceSdlGraphicsManager::copyRectToOverlay(const void *buf, int pitch, in
#pragma mark -
void SurfaceSdlGraphicsManager::setMouseCursor(const void *buf, uint w, uint h, int hotspotX, int hotspotY, uint32 keyColor, bool dontScale, const Graphics::PixelFormat *format) {
+ bool formatChanged = false;
+
if (format) {
#ifndef USE_RGB_COLOR
assert(format->bytesPerPixel == 1);
#endif
+ if (format->bytesPerPixel != _cursorFormat.bytesPerPixel) {
+ formatChanged = true;
+ }
_cursorFormat = *format;
} else {
+ if (_cursorFormat.bytesPerPixel != 1) {
+ formatChanged = true;
+ }
_cursorFormat = Graphics::PixelFormat::createFormatCLUT8();
}
@@ -1854,26 +1862,32 @@ void SurfaceSdlGraphicsManager::setMouseCursor(const void *buf, uint w, uint h,
_cursorDontScale = dontScale;
- if (_mouseCurState.w != (int)w || _mouseCurState.h != (int)h) {
+ if (_mouseCurState.w != (int)w || _mouseCurState.h != (int)h || formatChanged) {
_mouseCurState.w = w;
_mouseCurState.h = h;
- if (!w || !h) {
- return;
- }
-
if (_mouseOrigSurface) {
SDL_FreeSurface(_mouseOrigSurface);
if (_mouseSurface == _mouseOrigSurface) {
_mouseSurface = nullptr;
}
+
+ _mouseOrigSurface = nullptr;
+ }
+
+ if ((formatChanged || _cursorFormat.bytesPerPixel == 4) && _mouseSurface) {
+ SDL_FreeSurface(_mouseSurface);
+ _mouseSurface = nullptr;
+ }
+
+ if (!w || !h) {
+ return;
}
if (_cursorFormat.bytesPerPixel == 4) {
- if (_mouseSurface != _mouseOrigSurface) {
- SDL_FreeSurface(_mouseSurface);
- }
+ assert(!_mouseSurface);
+ assert(!_mouseOrigSurface);
const Uint32 rMask = ((0xFF >> format->rLoss) << format->rShift);
const Uint32 gMask = ((0xFF >> format->gLoss) << format->gShift);
@@ -1881,6 +1895,8 @@ void SurfaceSdlGraphicsManager::setMouseCursor(const void *buf, uint w, uint h,
const Uint32 aMask = ((0xFF >> format->aLoss) << format->aShift);
_mouseSurface = _mouseOrigSurface = SDL_CreateRGBSurfaceFrom(const_cast<void *>(buf), w, h, format->bytesPerPixel * 8, w * format->bytesPerPixel, rMask, gMask, bMask, aMask);
} else {
+ assert(!_mouseOrigSurface);
+
// Allocate bigger surface because AdvMame2x adds black pixel at [0,0]
_mouseOrigSurface = SDL_CreateRGBSurface(SDL_SWSURFACE | SDL_RLEACCEL | SDL_SRCCOLORKEY | SDL_SRCALPHA,
_mouseCurState.w + 2,
@@ -2069,7 +2085,7 @@ void SurfaceSdlGraphicsManager::blitCursor() {
dstPtr += _mouseOrigSurface->pitch - w * 2;
}
- if (sizeChanged) {
+ if (sizeChanged || !_mouseSurface) {
if (_mouseSurface)
SDL_FreeSurface(_mouseSurface);
More information about the Scummvm-git-logs
mailing list