[Scummvm-git-logs] scummvm master -> 6208f8807ec12303e3bd99bb6ddc83db39ffac43

[fracturehill]

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:
6208f8807e AGS: Properly pre-scale surface before blit


Commit: 6208f8807ec12303e3bd99bb6ddc83db39ffac43
    https://github.com/scummvm/scummvm/commit/6208f8807ec12303e3bd99bb6ddc83db39ffac43
Author: Kaloyan Chehlarski (strahy at outlook.com)
Date: 2023-11-23T00:44:19+02:00

Commit Message:
AGS: Properly pre-scale surface before blit

Fixed two different errors that were causing scaled blitting
to read out of bounds.

Changed paths:
    engines/ags/lib/allegro/surface.cpp


diff --git a/engines/ags/lib/allegro/surface.cpp b/engines/ags/lib/allegro/surface.cpp
index 57b189b6083..2d0748ed942 100644
--- a/engines/ags/lib/allegro/surface.cpp
+++ b/engines/ags/lib/allegro/surface.cpp
@@ -241,7 +241,7 @@ void BITMAP::stretchDraw(const BITMAP *srcBitmap, const Common::Rect &srcRect,
 	// We need to use Surface::scale, since ManagedSurface _always_ respects the source alpha, and thus skips transparent pixels
 	Graphics::ManagedSurface stretched(cropped.rawSurface().scale(dstRect.width(), dstRect.height()), DisposeAfterUse::YES);
 	BITMAP temp(&stretched);
-	auto optimizedArgs = DrawInnerArgs(this, &temp, srcRect, dstRect, skipTrans, srcAlpha, false, false, -1, -1, -1, true);
+	auto optimizedArgs = DrawInnerArgs(this, &temp, stretched.getBounds(), dstRect, skipTrans, srcAlpha, false, false, -1, -1, -1, false);
 	
 #ifdef SCUMMVM_NEON
 	if (_G(simd_flags) & AGS3::Globals::SIMD_NEON) {