From 9ae02c978c9131b5448293a7ee791025bfc3f202 Mon Sep 17 00:00:00 2001
From: Walter van Niftrik
Date: Tue, 17 Feb 2009 14:13:42 +0000
Subject: SCI: Added a bounds check on the index when setting palette.

svn-id: r38407
---
 engines/sci/gfx/gfx_driver.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'engines')

diff --git a/engines/sci/gfx/gfx_driver.cpp b/engines/sci/gfx/gfx_driver.cpp
index e96caa19e8..e5efd9d586 100644
--- a/engines/sci/gfx/gfx_driver.cpp
+++ b/engines/sci/gfx/gfx_driver.cpp
@@ -341,6 +341,12 @@ scummvm_set_pointer(struct _gfx_driver *drv, gfx_pixmap_t *pointer) {
 
 static int
 scummvm_set_palette(struct _gfx_driver *drv, int index, byte red, byte green, byte blue) {
+	if (index < 0 || index > 255)
+	{
+		GFXERROR("Attempt to set invalid palette entry %d\n", index);
+		return GFX_ERROR;
+	}
+
 	byte color[] = {red, green, blue, 255};
 	g_system->setPalette(color, index, 1);
 	return GFX_OK;
-- 
cgit v1.2.3