[PR #9320] Fix NRE in DisposeAsyncCore #12062

Closed
opened 2026-02-07 06:49:23 +03:00 by OVERLORD · 0 comments
Owner

Original Pull Request: https://github.com/jellyfin/jellyfin/pull/9320

State: closed
Merged: Yes


Changes
Fixes NRE in DisposeAsyncCore when _sessionManager is null (appears to be possible, happens when a broken plugin prevents startup)

An unhandled exception of type 'System.NullReferenceException' occurred in System.Private.CoreLib.dll: 'Object reference not set to an instance of an object.'
   at Emby.Server.Implementations.ApplicationHost.<DisposeAsyncCore>d__126.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Emby.Server.Implementations/ApplicationHost.cs:line 1190
   at Emby.Server.Implementations.ApplicationHost.<DisposeAsync>d__125.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Emby.Server.Implementations/ApplicationHost.cs:line 1152
   at Jellyfin.Server.Program.<StartServer>d__11.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Jellyfin.Server/Program.cs:line 260
   at Jellyfin.Server.Program.<StartApp>d__10.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Jellyfin.Server/Program.cs:line 177
   at Jellyfin.Server.Program.<Main>(String[] args)

Issues

None

**Original Pull Request:** https://github.com/jellyfin/jellyfin/pull/9320 **State:** closed **Merged:** Yes --- <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** Fixes NRE in DisposeAsyncCore when _sessionManager is null (appears to be possible, happens when a broken plugin prevents startup) ``` An unhandled exception of type 'System.NullReferenceException' occurred in System.Private.CoreLib.dll: 'Object reference not set to an instance of an object.' at Emby.Server.Implementations.ApplicationHost.<DisposeAsyncCore>d__126.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Emby.Server.Implementations/ApplicationHost.cs:line 1190 at Emby.Server.Implementations.ApplicationHost.<DisposeAsync>d__125.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Emby.Server.Implementations/ApplicationHost.cs:line 1152 at Jellyfin.Server.Program.<StartServer>d__11.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Jellyfin.Server/Program.cs:line 260 at Jellyfin.Server.Program.<StartApp>d__10.MoveNext() in /home/mbr/Desktop/projects/jellfyin/new-develop/jellyfin/Jellyfin.Server/Program.cs:line 177 at Jellyfin.Server.Program.<Main>(String[] args) ``` **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> None
OVERLORD added the pull-request label 2026-02-07 06:49:23 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#12062