(pln_equip): Do not abandon sector for paradrop, drop or fly.
Closes #982226

(tend): Send a bulletin to the ally when tending items to an ally.
Closes #929398.

(kill_cmd): Add authority check for deities before allowing a kill.
Closes #1215088.  Add banned host check before allowing a kill.

(natbyname): Do not look up a non-active country.
Previously a blank country name or deleted country name would
return a pointer to non-active countries.
This creates a security problem where some one can successfully
login into the server without country or password.  Closes #1219569.

(ship_chr_flags):  Add internal flags (M_XLIGHT, M_CHOPPER, M_UNIT)
to ship_chr_flags[].

(tend_land): Allow spy units to be tended to and from sub
with no land unit carrying capability.  Closes #1313537.

(dchrstr, pchrstr, rptstr): New member d_uid, p_uid, r_uid.
(dchr, bigcity_dchr, pchr, rpt): Initialize it.
(dchr_ca, pchr_ca, rpt_ca): New selector uid for it.

(ef_write): Don't memcpy() onto itself; behavior's undefined.

(multifire): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(player_coms): Document second argument of launch.

(laun, launch_as, launch_missile, launch_sat): When an argument is
bad, do not loop prompting, just fail.  That's what the vast majority
of commands do.

(launch_as, launch_missile, launch_sat): Don't zap the target
argument.  If the target is given as argument, launch all missiles at
that target.  Previously, missiles after the first prompted for
another target.

(buil): When an argument is bad, do not offer to list types, just
print how to list them.

(buil): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(getcontact,putcontact): Generalize the interface back to an int.
Add range checks to prevent unexpected results.

(buil): Remove N_NEUT check.  It should have been
removed when the opt_NEUTRON was removed.

(plurize): Fix the max. string length calculation for the 'y' case.
It was off by one because 'y' is removed from the string.

(nat_ca): Add nat_contact to nat_ca[].

(cons_choose): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(ef_ptr, ef_read): Undo rev. 1.52 for these two, because there are
callers that rely on these functions to fail for bad ID.

(add): When an argument is bad, do not loop prompting, just fail.
That's what the vast majority of commands do.

(add): Do not silently truncate country number argument.

(add): Diagnostics were confusing because they lacked context.

The previous revision accidentally deleted commented out code.
Restore.

(getcontact, putcontact, nat_contact): Change nat_contact
to unsigned char to reduce the space required for natstr.

(prod_nat): Remove non-functional aging code for nat_contact.
Remove nat_contact debugging from server log.

(update_main, prod_nat): Move aging code for nat_contact
from update_main to prod_nat.

(agecontact): Remove the opt_LOSE_CONTACT, the calling
procedure update_main() already has opt_LOSE_CONTACT check.

(getcontact, putcontact): Remove the bit shifting that was in the
function to save space.  opt_LOSE_CONTACT changes negated
any space saving.

(cens, desi, dump, do_desi, doland, file_sct_init, grow_continents)
(grow_islands, set_coastal, update_coastal_flag, coast_sea_to_land)
(coast_land_to_sea): Determine sct_coastal flag when the sector type
is set.  Old method only updated the sct_coastal flag when doing
cens() or a dump().  Other uses of sct_coastal could get stale values,
including selector coastal.  Rework set_coastal() into NEW functions
update_coastal_flag(), coast_sea_to_land(), coast_land_to_sea() and
set_coastal().  Fix a bug where the deity designation would not
properly set the coastal flag.  Fix a bug where the one sector lake
would not properly compute coastal flag.  Fix a bug that makes
fairland treat the last sector it adds to an island as non-coastal.

(file_sct_init, main, fl_sct_init, write_sects): Assign sct_timestamp with the
time the sector was created in files and fairland programs.

(add): Convert bridge spans or bridge towers into sea sectors instead the
wilderness sectors when wiping sectors.

(cens, cens_hdr): Print the actual old_owner instead of just a star
when different.  This is change the format for the census output.

(plurize): Change variable name max_len to size to improve readability.

(plurize): Fix initial length check to check for zero length of buf
not non-zero lengths of buf.

(plurize): Change variable name size to len to improve readability.

Update patch for src/lib/commands/mfir.c:1.26,src/server/main.c :1.79,include/prototypes.h:1.98

(fuel, load, prod, max_population, shp_nav_one_sector,
nav_loadship, upd_buildeff): Identify BIG_CITY
(IS_BIG_CITY) by using packing type of UPKG instead of
opt_BIG_CITY and sector type of SCT_CAPITAL.

(plurize): Fix the strncat() len parameter to prevent an overflow of buf.

(numstr, esplur, splur, iesplur, plurize): Move the prototype defintions from
misc.h to prototypes.h.  Remove unneccessary headers in plur.c.

(budg): Create the capital/city name from the d_name field instead of
hard coding the strings.

(plurize): New.

(opt_NO_LCMS, opt_NO_HCMS, opt_NO_OIL, Options,
buil, show_bridge, show_tower, nullify_objects):
Remove NO_LCMS, NO_HCMS, and NO_OIL options, not used.
Functionality can be emulated by modifiying the characteristics files.

(opt_NEUTRON, buil, denotate, show_nuke_capab, show_nuke_build,
Options): Remove opt_NEUTRON option.  The option can be emulated
by manipulating the flags in a nuke characteristics file.

(dchr_ca, packing, empfile): Add packing symbol table.

(sect_ca, ship_ca, land_ca, plague_stages, empfile):
Add plague stages symbol table.

(treaty_ca, loan_ca, agreement_statuses, empfile):
Add agreement status symbol table.
Combine treatystatuses (TS_FREE, TS_SIGNED and TS_PENDING)
and loan statuses (LS_FREE, LS_SIGNED and LS_PENDING)
into a single agreement status.

(pchr_ca, level, empfile): Add packing symbol table.

(output): Remove C_FLUSH from output() as C_FLUSH is dealt with
by servercmd() and output() is not called from servercmd() with a
code of C_FLUSH.

(xunsymbol): Fix xunsymbol() to parse multiple symbols for a
symbol_set (NSC_BITS).

(xundump): Add prototypes.h to get a prototype for
xundump().

(servercmd): Remove newline for C_INFORM, C_FLUSH,
C_REDIR, C_PIPE and C_EXECUTE.
Fixes the problem created when the newline was left in io_gets().

(buil, vers, opt_NONUKES, show_nuke_build, show_nuke_capab):
Remove opt_NONUKES options.  A nuke characteristics file can
be created with no entries to create the emulate the removed option.

(dchr_ca): prd indexes product table.

(EF_MISSION_FLAGS, EF_MISSIONS, mission_flags, missions): Rename, as
these are not flags.

(init_server, ef_load, xundump): Add ability to customize game
configurations.  The custom game configurations are loaded via
text files in the data directory.  Closes #752881.

(io_gets, output): Remove the removal of \n in io_gets() and the
readdition of it in output().

(servercmd, io_gets, output): Fix client to read lines longer than
1024 bytes from the server.  Closes #1223255.

(xdump): Add EF_MAX check for table selection by number.
Fixes a crash when selecting a table over EF_MAX.

(plane_flags, plane_ca): Add plane flags table.

(retreat_flags, ship_ca, land_ca, plane_ca): Add retreat flags table.

(nation_flags, nation_relations, nat_ca): Add nation flags and
nation relations tables.

(empfile_ca): Remove flags and file name fields.
They are internal and not needed by the client.

(start_server, empth_start): Passed uninitialized sa_mask to
sigaction().  This used to broken only when SA_SIGINFO wasn't defined,
until main.c rev. 1.46 and pthread.c rev. 1.10 broke it
unconditionally.

Fix bad line break.

(ef_init_chr): Sentinel names are now null, simplify sentinel check.

(init_server): Remove ef_load() call.
New function not ready yet, incorrectly
checked-in.

(itm_maxno): Remove itm_maxno, it is not used.

(nullify_objects): Fix sentinel row checks for plchr, lchr and mchr loops.

(shp_maxno): Variable's value is well-known constant N_MAXSHIPS + 1.
Useless, remove.  Users changed to check mchr's sentinel, because
that's clearer than comparing the index to N_MAXSHIPS or sizeof(mchr)
/ sizeof(*mchr).

(mchr): Change sentinel's m_name to NULL.

(pln_maxno): Variable's value is well-known constant N_MAXPLANE + 1.
Useless, remove.  Users changed to check plchr's sentinel, because
that's clearer than comparing the index to N_MAXPLANE or sizeof(plchr)
/ sizeof(*plchr).

(plchr): Change sentinel's pl_name to NULL.

(pstat): Remove inappropriate plane sanity check; should be done
elsewhere.

(lnd_maxno): Variable's value is well-known constant N_MAXLAND + 1.
Useless, remove.  Users changed to check lchr's sentinel, because
that's clearer than comparing the index to N_MAXLAND or sizeof(lchr) /
sizeof(*lchr).

(lchr): Change sentinel's l_name to NULL.

(carg, lcarg, fort, land, ldump, lsta, orde, payo, retr, lretr)
(cmd_sail_ship, cmd_unsail_ship, show_sail, sdump, shi, sstat, work):
Remove inappropriate ship/land sanity checks; should be done
elsewhere.

(show_nuke_build, show_nuke_capab): Simplify.

(nuk_maxno): Variable's value is well-known constant N_MAXNUKE + 1.
Useless, remove.  Users changed to check nchr's sentinel, because
that's clearer than comparing the index to N_MAXNUKE or sizeof(nchr) /
sizeof(*nchr).

(nchr): Change sentinel's n_name to NULL.

(buil): Use typematch() to search for ship, plane, land unit and nuke
type.  The old code didn't work correctly when "sbb  super battleship"
came before "sb   submarine" in mchr[].

(arm, tran_nuke): Lookup of the nuke name was broken by nuke.h
rev. 1.10.  Replace by typematch().

(arm): Simplify search for nuke at sector.

(prd_maxno): Variable's value is well-known constant P_MDUST + 2.
Useless, remove.  Users changed to check pchr's sentinel, because
that's clearer than comparing the index to P_MDUST or sizeof(pchr) /
sizeof(*pchr).

(pchr): Change sentinel's p_name and p_sname to NULL.  Sentinel
checks changed.

(empfile,mission_flags,ship_ca,land_ca,plane_ca):  Add mission flags table.

(nullify_objects): Fix option NO_OIL.  The attempt to remove ships
with capability M_OIL from mchr[] actually replaced them with
sentinels.  Just clear the capability instead.

(xdopt, xdver): Simplify the previous rev.

(xdump): Support selecting a dump by table number in addition to name.

(ef_ptr, ef_read): Oops on bad ID.  Some callers rely on these
functions to fail at the end of the table.  To support that, the first
ID beyond the end is not considered bad.

(ef_ensure_space): Oops on bad ID.

(NSC_BITS): Remove the trailing comma as it is last row.

(install_service):  Use the compiled in default for location of econfig
when the -e is not specified while creating a service.

(xdhdr,xdhdr1,xditem,xdmeta,xdopt,xdver,xdfldnam):
Remove the column header.
Not need as the xdump meta is functional.

(xdopt,xdver,xdump): Add meta data for opt and ver.

(xdmeta): Fix the printing of the number of rows dumped.
Incorrectly included the skipped rows.

(xdmeta): Don't dump meta data for stuff that is not dumped by
xdflds().

(NSC_BITS): New.
(meta_flags): Update for it.
(ship_ca, mchr_ca, plane_ca, plchr_ca, land_ca, lchr_ca, nchr_ca)
treaty_ca, nat_ca, empfile_ca, mdchr_ca): Use it.

(symbol_ca): Move value selector to beginning of array.

(meta_flags): Downcase names.

(mission_short_name):   Remove mission_short_name, not used.

(nullify_objects): Patch intrchr[] according to opt_NO_LCMS,
opt_NO_HCMS.  This fixes output of `show se b', which used not to
reflect these options.
(improve): Simplify.

(xdmeta): Fix rev. 1.36.

(UNMAPPED_INIT, UNMAPPED_CACHE, ARRAY_INIT, ARRAY_TABLE, PTR_INIT)
(PTR_CACHE): Rename.

(ARRAY_CACHE): New.
(empfile): Use it to set csize of elements EF_SHIP_CHR, EF_PLANE_CHR,
EF_LAND_CHR, EF_NUKE_CHR.
(ef_init_chr): Now that empfile[]'s initializer sets csize, remove the
temporary csize initialization here.  The value wasn't correct, but
better than nothing.

(perform_mission): Do not pass the sub-owner to nreport().  News
doesn't show it, but selectors do.  Reported by Doug Wescott.

(nreport, ncache): Catch errors like the above.  Catch bad event
argument while we're at it.

(xdopt, xdver): Fix missing footer.

(xdmeta): Add NULL ca check.
Fixes a crash for xdump meta pow.

(ichr_ca, pchr_ca, sect_ca, dchr_ca, NSC_GENITEM, ship_ca, mchr_ca)
plane_ca, plchr_ca, land_ca, lchr_ca, nuke_ca, nchr_ca, treaty_ca)
loan_ca, news_ca, lost_ca, commodity_ca, trade_ca, nat_ca)
(empfile_ca): Move row index selector to beginning of array and set
its table.  Set table for NSC_SITYPE(i_type), NSC_NATID, NSC_TYPEID
selectors.
(news_ca): Set selector action's table.
(commodity_ca, trade_ca): Set selector maxbidder's table.

(trtstr): New members ef_type and trt_uid.
(do_treaty): Initialize them.
(treaty_ca): New selector uid.

(ef_ptr): Null cache can't happen.

(ef_open): EFF_STATIC and cache not null can't happen.

(ef_open): Leave check of fd to do_write().

(empfile_ca): Rename selector file_name to file, to match the member
name.

(ichr): Define without array size, like the other characteristics
tables.

Doc fix.

(diroff, dirch, routech): Declare with array size.  Define without.
This requires moving routech[] declaration from map.h to path.h.  Get
rid of s_char.

(ep): Global variable pointing to sector cache.  Gross.  Remove.

(bp_neighbors, bp_lbcost, pathcost): Replace pointer arithmetic by
array subscript.

C_EXECUTE security.

(coun): Last access used to report last login time.
Counter-intuitive.  Report last connection time range instead.
Unfortunately, this requires a change in output format.

(coun): Report on all countries even to mortals.  This used to be
done, and was most probably changed because it gave away too much
information.  Carefully limit the information given to mortals, as
follows.  Limit last access information to whether deities or allies
are logged on right now, no longer report connection time for deities
that are not logged on.  Remove column time for mortals, rename it to
BTU for deities.  Report status, but with less detail if HIDDEN is
enabled.

(buy, check_market, trad, check_trade): Remove the following virtually
undocumented feature: When a buyer didn't have enough money at the
time of the trade, the seller *automatically* granted a high-interest,
medium term loan for up to 90% of the price.  This is just a silly
trap for unwary sellers and buyers alike.
(N_FIN_TROUBLE, N_CREDIT_JUNK): Unused, remove.
(rpt): Update accordingly.

(empfile): Rename member ef_uid to uid for consistency with other
members.

(empfile): Reorder members for clearer documentation.

(EFF_IMMUTABLE): New.
(EFF_OPEN): Remove, use ~EFF_IMMUTABLE.

(empfile): Don't initialize member size to a useless value for EF_MAP
and EF_BMAP.  ef_init() writes the real value.

(ef_init_chr): Add a NULL check to sentinel check.

(ef_init): Fix check for sentinel.

(xdump): Use xditem() for all empfile tables.
(xdchr, xdmeta): Specialize for meta-tables, rename.

(meta_flags, meta_type): Change sentinel name to NULL to match the
other symbol tables.