summaryrefslogtreecommitdiffstats
path: root/core/divelist.c
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2020-04-13 12:17:43 +0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2020-04-13 08:52:48 -0700
commit103951bc78485beb615feafd1f3d8e7aac3120ea (patch)
treead0fc3b8ab2f7258c5658b56f5c76a11f388d7fe /core/divelist.c
parent3d4412ad1a346d80959cd78395703636d9127699 (diff)
downloadsubsurface-103951bc78485beb615feafd1f3d8e7aac3120ea.tar.gz
divelist: when removing/adding a dive unregister/register fulltext
This fixes a crash: when the undo commands removed a dive from the list, the fulltext cache was not cleared. If now the divelist is reset and then the undo-command deleted, deletion of the owned dive tries to remove it's fulltext cache, which doesn't exist anymore. For reasons of symmetry, when readding the dive, its fulltext has to be registered. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/divelist.c')
-rw-r--r--core/divelist.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/core/divelist.c b/core/divelist.c
index 750eb42db..994170570 100644
--- a/core/divelist.c
+++ b/core/divelist.c
@@ -753,6 +753,9 @@ struct dive *unregister_dive(int idx)
struct dive *dive = get_dive(idx);
if (!dive)
return NULL; /* this should never happen */
+ /* When removing a dive from the global dive table,
+ * we also have to unregister its fulltext cache. */
+ fulltext_unregister(dive);
remove_from_dive_table(&dive_table, idx);
if (dive->selected)
amount_selected--;