From: Eric S. Raymond <esr@thyrsus.com>
Date: Sun, 12 Aug 2018 15:59:55 +0000 (-0400)
Subject: Address GitLab issue #1: Crash when issuing DOCK command next to inhabited planet
X-Git-Tag: 2.3~2
X-Git-Url: https://jxself.org/git/?a=commitdiff_plain;h=e66fcee3307eb7795a9c2c2cf606c451e603698c;p=super-star-trek.git

Address GitLab issue #1: Crash when issuing DOCK command next to inhabited planet

Actually this would have happened anytime docking with no starbase in sector.
---

diff --git a/NEWS b/NEWS
index 47458c9..eac2608 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,8 @@
 			SST2K project news.
 
+Repository head:
+    Crash bug fix when docking with no starbase in sector.
+
 2.2 @ 2017-03-01
     Added color. Added BSD Trek 'CAPTURE' and 'CLOAK' (from Tom Almy's
     2013 changes).  Many bug fixes.
diff --git a/sst.py b/sst.py
index 945017f..bd0582e 100755
--- a/sst.py
+++ b/sst.py
@@ -86,8 +86,6 @@ class Coord:
         return self.i >= 0 and self.i < QUADSIZE and self.j >= 0 and self.j < QUADSIZE
     def invalidate(self):
         self.i = self.j = None
-    def is_valid(self):
-        return self.i != None and self.j != None
     def __eq__(self, other):
         return other != None and self.i == other.i and self.j == other.j
     def __ne__(self, other):
@@ -3855,7 +3853,7 @@ def dock(verbose):
     if game.inorbit:
         prout(_("You must first leave standard orbit."))
         return
-    if not game.base.is_valid() or abs(game.sector.i-game.base.i) > 1 or abs(game.sector.j-game.base.j) > 1:
+    if game.base is None or abs(game.sector.i-game.base.i) > 1 or abs(game.sector.j-game.base.j) > 1:
         prout(crmshp() + _(" not adjacent to base."))
         return
     if game.iscloaked:
@@ -4491,7 +4489,7 @@ def mayday():
             # found one -- finish up
             game.sector = w
             break
-    if not game.sector.is_valid():
+    if game.sector is None:
         prout(_("You have been lost in space..."))
         finish(FMATERIALIZE)
         return
@@ -4671,7 +4669,7 @@ def orbit():
     if damaged(DWARPEN) and damaged(DIMPULS):
         prout(_("Both warp and impulse engines damaged."))
         return
-    if not game.plnet.is_valid():
+    if game.plnet is None:
         prout("There is no planet in this sector.")
         return
     if abs(game.sector.i-game.plnet.i)>1 or abs(game.sector.j-game.plnet.j)>1: