]> git.pond.sub.org Git - empserver/commit
Fix seqno mismatch in fly()
authorMarkus Armbruster <armbru@pond.sub.org>
Sat, 12 Jul 2008 14:56:30 +0000 (10:56 -0400)
committerMarkus Armbruster <armbru@pond.sub.org>
Sat, 12 Jul 2008 18:51:18 +0000 (14:51 -0400)
commit42d9475d894f6779dab17655cd5082b66b9acac9
treece42891338e053c34b1accb0aedcd7f37a9f7019
parent89d168cd65a3496f06ec8e9c0963d91a91896d47
Fix seqno mismatch in fly()

fly() reads the carrier, then passes it to pln_dropoff(), which writes
it back.  fly() also calls pln_oneway_to_carrier_ok(), which updates
the carrier when its plane summary information is incorrect.

The old code called it between reading the carrier and passing it to
pln_dropoff().  This made pln_dropoff() wipe out the plane summary
update, and triggered a seqno mismatch oops.  Broken by introduction
of pln_oneway_to_carrier_ok() in commit 1127762c, v4.2.17.

Fix by reading the carrier right before passing it to pln_dropoff().
src/lib/commands/fly.c