From 18c8a920d2f357c3f83eb1aa228d2432e333a399 Mon Sep 17 00:00:00 2001
From: KUDr <KUDr@openttd.org>
Date: Sat, 21 Jul 2007 14:01:12 +0000
Subject: (svn r10647) -Fix: AutoPtrT::operator =() didn't delete old object

---
 src/misc/autoptr.hpp | 8 ++++++++
 1 file changed, 8 insertions(+)

(limited to 'src/misc')

diff --git a/src/misc/autoptr.hpp b/src/misc/autoptr.hpp
index edb7c10c6..8b0d920f5 100644
--- a/src/misc/autoptr.hpp
+++ b/src/misc/autoptr.hpp
@@ -81,8 +81,16 @@ public:
 	/** assignment operator */
 	FORCEINLINE AutoPtrT& operator = (const AutoPtrT& src)
 	{
+		/* Save original pointer and replace it with the given one to avoid recursive calls. */
+		T* p = m_p;
 		m_p = src.m_p;
+
 		if (m_p != NULL) src.m_p = NULL;
+
+		if (p != NULL) {
+			/* Now we can safely delete the old one. */
+			delete p;
+		}
 		return *this;
 	}
 
-- 
cgit v1.2.3-70-g09d2