bomb launch interdiction: Fix crash on bombs missing target
authorMarkus Armbruster <armbru@pond.sub.org>
Sun, 27 Dec 2020 11:43:19 +0000 (12:43 +0100)
committerMarkus Armbruster <armbru@pond.sub.org>
Sun, 17 Jan 2021 20:24:28 +0000 (21:24 +0100)
commite312e4f8d6eea49423722c87b7f368e42b1086be
tree52f0aa1ac6d8d906ae35e581f045c66b09cd39f0
parent77d3da4f098fbf60b62051b25c9a1a8d3efaefb2
bomb launch interdiction: Fix crash on bombs missing target

plane damage() prints @noisy with snprintf() even when it's null.  It
doesn't actually use the output then.  Some systems (GNU, Windows)
deal gracefully with printing null strings, others crash.  @noisy is
null when bombers or missiles miss and do collateral damage.  Affects
bomb, launch, and interdiction missions.  Broken in commit 820d755e5
"subs: Change pln_damage()'s parameter noisy to string prefix",
v4.3.33.

Fix by guarding the snprintf().

Signed-off-by: Markus Armbruster <armbru@pond.sub.org>
src/lib/subs/plnsub.c