Without the patch, there is a crash if the dive does not have a
location.

Signed-off-by: Miika Turkia <[email protected]>
---
 liquivision.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/liquivision.c b/liquivision.c
index 7126c9b..2f1bb00 100644
--- a/liquivision.c
+++ b/liquivision.c
@@ -137,8 +137,12 @@ static void parse_dives (int log_version, const unsigned 
char *buf, unsigned int
                } else if (place_len) {
                        location = strndup(buf + ptr + len + 4, place_len);
                }
-               dive->dive_site_uuid = create_dive_site(location);
-               free(location);
+
+               /* Store the location only if we have one */
+               if (len || place_len) {
+                       dive->dive_site_uuid = create_dive_site(location);
+                       free(location);
+               }
 
                ptr += len + 4 + place_len;
 
-- 
2.1.0

_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to