]> 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>
Sun, 13 Jul 2008 12:07:38 +0000 (08:07 -0400)
commit6aefeea2a168338a6eb13e9d096c4373f141a6a9
tree68cd6918cf804fc276ff9854967196b73d8f2b53
parent1713f2d3039dd40593a617ebafab32c5366c5445
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().
(cherry picked from commit 42d9475d894f6779dab17655cd5082b66b9acac9)
src/lib/commands/fly.c