Skip to content

fix stats sync in multiplayer#7486

Merged
Goober5000 merged 1 commit into
scp-fs2open:masterfrom
Goober5000:fix/stats_alltime_kills
May 30, 2026
Merged

fix stats sync in multiplayer#7486
Goober5000 merged 1 commit into
scp-fs2open:masterfrom
Goober5000:fix/stats_alltime_kills

Conversation

@Goober5000

Copy link
Copy Markdown
Contributor

A quick bugfix, thanks to Claude. Make sure the end of the loop is correctly offset from the start.

A quick bugfix, thanks to Claude.  Make sure the end of the loop is correctly offset from the start.
@Goober5000 Goober5000 added fix A fix for bugs, not-a-bugs, and/or regressions. multi A feature or issue related to the multiplayer code. labels May 30, 2026

@JohnAFernandez JohnAFernandez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to fix it. I don't think we were aware of this bug, but it might cause the sever to crash or get garbage packet data if the player joining had any kills to send.

@JohnAFernandez

Copy link
Copy Markdown
Contributor

And it shouldn't need a Multi version bump or any other kind of handling, as it fixes the packet alignment to be the same on client and server.

@Goober5000

Goober5000 commented May 30, 2026

Copy link
Copy Markdown
Contributor Author

Cool. 👍 Thanks for the review!

@Goober5000 Goober5000 merged commit 14b0edc into scp-fs2open:master May 30, 2026
20 checks passed
@Goober5000 Goober5000 deleted the fix/stats_alltime_kills branch May 30, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A fix for bugs, not-a-bugs, and/or regressions. multi A feature or issue related to the multiplayer code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants