Cleanup: split out free_dive() function from delete_single_dive()

Currently, we can only delete dives that are indexed in the main
dive table. In the future, we will have to delete dives outside
of this table (e.g. for undo). Therefore, split out the free_dive()
function from delete_single_dive(), which takes an index into
the main dive table.

In the process, adopt the dive freeing-code from clear_dive(),
which frees more data than the code in delete_single_dive().
This potentially fixes a memory-leak.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>

1 files changed, 18 insertions, 5 deletions

diff --git a/core/dive.c b/core/dive.c
index 0c185efa5..83789f686 100644
--- a/core/dive.c
+++ b/core/dive.c
@@ -528,11 +528,7 @@ static void copy_tl(struct tag_entry *st, struct tag_entry *dt)
 		*_dptr = 0;                              \
 	}
 
-/* copy_dive makes duplicates of many components of a dive;
- * in order not to leak memory, we need to free those .
- * copy_dive doesn't play with the divetrip and forward/backward pointers
- * so we can ignore those */
-void clear_dive(struct dive *d)
+static void free_dive_structures(struct dive *d)
 {
 	if (!d)
 		return;
@@ -550,6 +546,23 @@ void clear_dive(struct dive *d)
 		free((void *)d->cylinder[i].type.description);
 	for (int i = 0; i < MAX_WEIGHTSYSTEMS; i++)
 		free((void *)d->weightsystem[i].description);
+}
+
+void free_dive(struct dive *d)
+{
+	free_dive_structures(d);
+	free(d);
+}
+
+/* copy_dive makes duplicates of many components of a dive;
+ * in order not to leak memory, we need to free those .
+ * copy_dive doesn't play with the divetrip and forward/backward pointers
+ * so we can ignore those */
+void clear_dive(struct dive *d)
+{
+	if (!d)
+		return;
+	free_dive_structures(d);
 	memset(d, 0, sizeof(struct dive));
 }