From faf4c1fb6012941134046db14370f60748184b05 Mon Sep 17 00:00:00 2001
From: Chris Young <chris@unsatisfactorysoftware.co.uk>
Date: Sat, 19 Nov 2016 21:08:41 +0000
Subject: Free and clear icondata to avoid a potential double-free

---
 frontends/amiga/bitmap.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'frontends/amiga/bitmap.c')

diff --git a/frontends/amiga/bitmap.c b/frontends/amiga/bitmap.c
index 7609d9451..124c116d5 100644
--- a/frontends/amiga/bitmap.c
+++ b/frontends/amiga/bitmap.c
@@ -730,14 +730,15 @@ void ami_bitmap_set_title(struct bitmap *bm, const char *title)
 	bm->title = strdup(title);
 }
 
-ULONG *ami_bitmap_get_icondata(struct bitmap *bm)
+void ami_bitmap_set_icondata(struct bitmap *bm, ULONG *icondata)
 {
-	return bm->icondata;
+	bm->icondata = icondata;
 }
 
-void ami_bitmap_set_icondata(struct bitmap *bm, ULONG *icondata)
+void ami_bitmap_free_icondata(struct bitmap *bm)
 {
-	bm->icondata = icondata;
+	if(bm->icondata) FreeVec(bm->icondata);
+	bm->icondata = NULL;
 }
 
 bool ami_bitmap_is_nativebm(struct bitmap *bm, struct BitMap *nbm)
-- 
cgit v1.2.3