From 90dbcdb3c10eba860b4d0776d292b673cd136936 Mon Sep 17 00:00:00 2001
From: duncanspumpkin <duncans_pumpkin@hotmail.co.uk>
Date: Tue, 22 Nov 2016 19:36:24 +0000
Subject: [PATCH] Fix #4820. Fix banner signs build cost.

Issue was caused by an incorrect read (only 1 byte instead of 2) of the legacy banner data during object loading. Note this would have caused the scenery_tab_id and string table to possibly have corrupted data.
---
 src/object/BannerObject.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/object/BannerObject.cpp b/src/object/BannerObject.cpp
index 7bf92b92a4..fad952bf42 100644
--- a/src/object/BannerObject.cpp
+++ b/src/object/BannerObject.cpp
@@ -29,7 +29,7 @@ void BannerObject::ReadLegacy(IReadObjectContext * context, IStream * stream)
     stream->Seek(6, STREAM_SEEK_CURRENT);
     _legacyType.banner.scrolling_mode = stream->ReadValue<uint8>();
     _legacyType.banner.flags = stream->ReadValue<uint8>();
-    _legacyType.banner.price = stream->ReadValue<uint8>();
+    _legacyType.banner.price = stream->ReadValue<sint16>();
     _legacyType.banner.scenery_tab_id = stream->ReadValue<uint8>();
 
     GetStringTable()->Read(context, stream, OBJ_STRING_ID_NAME);