Constify arguments to getsmap.  constify and correct arguments to show_map.  Remove unneeded arguments.  Hardcode show to 1 as it's never actually turned off anyware.  Use C++ initializer for shiplocs, avoid uninitialized read caught by MSan.  Constify arguments to desshow. Constify arguments to compatibility.
Add testplan.  This needs to be captured in unit tests at some point, but for now at least write it down
Pass tmpdest as a parameter to do_trip rather than have it as a function static to avoid global constructor
Not happy with this little bit of hacking.  The upstream code seems to rely on the strings being handed in being longer, writable, and full of zeroes to work right.  The logic needs a bit of work and this at least works under ASan now
Make a generic rule for running that includes everything needed for MSan and ASan
Initialize sectypes directly rather than using bzero