aboutsummaryrefslogtreecommitdiffstats
path: root/uemis.c
diff options
context:
space:
mode:
authorGravatar glerch <guido.lerch@gmail.com>2015-09-10 20:54:03 +0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2015-09-10 12:44:12 -0700
commit682e23775c93a00804f71131342ee52aeacc6337 (patch)
tree711b55337aed763cac2ea228bdadce5eb85ac4a5 /uemis.c
parentdc02a7aa6db3ff4211bce42f48fcdc3722594322 (diff)
downloadsubsurface-682e23775c93a00804f71131342ee52aeacc6337.tar.gz
Uemis downloader: bug fix in uemis set dive location
Assuring that ds isn't NULL, had a few cases that made SubSurface crash. Have not investigated the root cause but it's likely that it is related to the Uemis specific code. Nevertheless, checking ds non NULL is certainly not a bad thing to do. Signed-off-by: glerch <guido.lerch@gmail.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'uemis.c')
-rw-r--r--uemis.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/uemis.c b/uemis.c
index 1c592832a..352c9a028 100644
--- a/uemis.c
+++ b/uemis.c
@@ -173,9 +173,11 @@ void uemis_set_divelocation(int divespot, char *text, double longitude, double l
while (hp) {
if (hp->divespot == divespot) {
struct dive_site *ds = get_dive_site_by_uuid(hp->dive_site_uuid);
- ds->name = strdup(text);
- ds->longitude.udeg = round(longitude * 1000000);
- ds->latitude.udeg = round(latitude * 1000000);
+ if (ds) {
+ ds->name = strdup(text);
+ ds->longitude.udeg = round(longitude * 1000000);
+ ds->latitude.udeg = round(latitude * 1000000);
+ }
}
hp = hp->next;
}