Fix a valgrind warning.

It is *not* a good idea to pass a reference to a list entry to List::remove.
Since List::remove will remove *all* occurances of that list entry, it will
also invaldiate the reference, resulting in invalid memory reads after the
entry has been removed from the list, when List::remove will continue to
search the rest of the list for more occurances of the same entry.

svn-id: r48225

1 files changed, 4 insertions, 3 deletions

diff --git a/engines/saga/events.cpp b/engines/saga/events.cpp
index 98f206b29d..0ed1f2b3d9 100644
--- a/engines/saga/events.cpp
+++ b/engines/saga/events.cpp
@@ -288,9 +288,10 @@ int Events::handleOneShot(Event *event) {
 		case kEventDisplay:
 			((TextListEntry *)event->data)->display = true;
 			break;
-		case kEventRemove:
-			_vm->_scene->_textList.remove(*((TextListEntry *)event->data));
-			break;
+		case kEventRemove: {
+			TextListEntry entry = *((TextListEntry *)event->data);
+			_vm->_scene->_textList.remove(entry);
+			} break;
 		default:
 			break;
 		}