From c9278d7dea8d020d0d85f2cb69bdee826ae27991 Mon Sep 17 00:00:00 2001
From: Erik Andresen <erik@vontaene.de>
Date: Mon, 26 May 2025 21:32:34 +0200
Subject: [PATCH] gipy: Avoid crash on negative array access

---
 apps/gipy/ChangeLog     |  2 ++
 apps/gipy/app.js        | 22 ++++++++++++----------
 apps/gipy/metadata.json |  2 +-
 apps/gipy/settings.js   | 11 ++++++++++-
 4 files changed, 25 insertions(+), 12 deletions(-)

diff --git a/apps/gipy/ChangeLog b/apps/gipy/ChangeLog
index b0445c161..17a701de4 100644
--- a/apps/gipy/ChangeLog
+++ b/apps/gipy/ChangeLog
@@ -141,3 +141,5 @@
 0.26: Add option to plot openstmap if installed
 
 0.27: Support for large paths (grid sizes > 65k)
+
+0.28: Avoid crash on negative array access
diff --git a/apps/gipy/app.js b/apps/gipy/app.js
index 659e12d98..7127ab132 100644
--- a/apps/gipy/app.js
+++ b/apps/gipy/app.js
@@ -34,6 +34,7 @@ var settings = Object.assign(
     power_lcd_off: false,
     powersave_by_default: false,
     sleep_between_waypoints: false,
+    keep_gps_alive: true
   },
   s.readJSON("gipy.json", true) || {}
 );
@@ -255,6 +256,7 @@ class Map {
   add_to_tile_image(img, absolute_tile_x, absolute_tile_y) {
     let tile_x = absolute_tile_x - this.first_tile[0];
     let tile_y = absolute_tile_y - this.first_tile[1];
+    if (tile_x < 0 || tile_y < 0) return; // FIXME: Negative Array index
     let side = img.getWidth() - 6;
 
     let thick = this.color[0] == 1;
@@ -609,16 +611,16 @@ class Status {
       );
 
       // disable gps when far from next point and locked
-      // if (Bangle.isLocked() && !settings.keep_gps_alive) {
-      //   let time_to_next_point =
-      //     (this.distance_to_next_point * 3.6) / settings.max_speed;
-      //   if (time_to_next_point > 60) {
-      //     Bangle.setGPSPower(false, "gipy");
-      //     setTimeout(function () {
-      //       Bangle.setGPSPower(true, "gipy");
-      //     }, time_to_next_point);
-      //   }
-      // }
+      if (Bangle.isLocked() && !settings.keep_gps_alive) {
+        let time_to_next_point =
+          (this.distance_to_next_point * 3.6) / settings.max_speed;
+        if (time_to_next_point > 60) {
+          Bangle.setGPSPower(false, "gipy");
+          setTimeout(function () {
+            Bangle.setGPSPower(true, "gipy");
+          }, time_to_next_point);
+        }
+      }
       let reaching_waypoint = this.path.is_waypoint(next_point);
       if (this.distance_to_next_point <= 100) {
         if (reaching_waypoint || !settings.sleep_between_waypoints) {
diff --git a/apps/gipy/metadata.json b/apps/gipy/metadata.json
index 91e371c16..04ff05caf 100644
--- a/apps/gipy/metadata.json
+++ b/apps/gipy/metadata.json
@@ -2,7 +2,7 @@
   "id": "gipy",
   "name": "Gipy",
   "shortName": "Gipy",
-  "version": "0.27",
+  "version": "0.28",
   "description": "Follow gpx files using the gps. Don't get lost in your bike trips and hikes.",
   "allow_emulator":false,
   "icon": "gipy.png",
diff --git a/apps/gipy/settings.js b/apps/gipy/settings.js
index 9562bc3c1..76e073f25 100644
--- a/apps/gipy/settings.js
+++ b/apps/gipy/settings.js
@@ -11,6 +11,7 @@
             power_lcd_off: false,
             powersave_by_default: false,
             sleep_between_waypoints: false,
+            keep_gps_alive: true
         },
         require("Storage").readJSON(FILE, true) || {}
     );
@@ -97,6 +98,14 @@
                 settings.sleep_between_waypoints = v;
                 writeSettings();
             }
-        }
+        },
+        "keep gps alive": {
+          value: !!settings.keep_gps_alive, // !! converts undefined to false
+          format: (v) => (v ? "Yes" : "No"),
+          onchange: (v) => {
+            settings.keep_gps_alive = v;
+            writeSettings();
+          },
+        },
     });
 })