Skip to content

fix: NullReferenceExceptions in spawn path#4067

Open
EmandM wants to merge 3 commits into
develop-2.0.0from
fix/null-reference-exceptions
Open

fix: NullReferenceExceptions in spawn path#4067
EmandM wants to merge 3 commits into
develop-2.0.0from
fix/null-reference-exceptions

Conversation

@EmandM

@EmandM EmandM commented Jul 2, 2026

Copy link
Copy Markdown
Member

Purpose of this PR

  • Fix null reference exceptions in NonAuthority object spawn when object creation fails
  • Update function naming to make behaviour clearer
  • Add CodeAnalysis tags to help identify future issues in this area
  • Add tests to catch this issue in the future
  • Update other serialization tests to run with the CMB service.
  • Fix potential editor memory leak in ILPP code

Jira ticket

n/a

Changelog

  • Fixed: Null reference exception thrown when NetworkObject fails to spawn on the non-authority.

Documentation

  • No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

  • Covered by existing automated tests
  • Covered by new automated tests

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?
  • Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

n/a

@EmandM EmandM requested a review from a team as a code owner July 2, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant