Added copy constructor and operator= to avoid blind copy of XPM pointer

leading to problems when double freed.

1 files changed, 15 insertions, 0 deletions

diff --git a/src/LineMarker.h b/src/LineMarker.h
index 0ac4d3155..c75a31230 100644
--- a/src/LineMarker.h
+++ b/src/LineMarker.h
@@ -43,9 +43,24 @@ public:
 		back = ColourDesired(0xff,0xff,0xff);
 		pxpm = NULL;
 	}
+	LineMarker(const LineMarker &) {
+		// Defined to avoid pxpm being blindly copied, not as real copy constructor
+		markType = SC_MARK_CIRCLE;
+		fore = ColourDesired(0,0,0);
+		back = ColourDesired(0xff,0xff,0xff);
+		pxpm = NULL;
+	}
 	~LineMarker() {
 		delete pxpm;
 	}
+	LineMarker &operator=(const LineMarker &) {
+		// Defined to avoid pxpm being blindly copied, not as real assignment operator
+		markType = SC_MARK_CIRCLE;
+		fore = ColourDesired(0,0,0);
+		back = ColourDesired(0xff,0xff,0xff);
+		pxpm = NULL;
+		return *this;
+	}
 	void RefreshColourPalette(Palette &pal, bool want);
 	void SetXPM(const char *textForm);
 	void SetXPM(const char * const *linesForm);