Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash fix: Guard against null hlsMuxer #104

Merged
merged 1 commit into from
Feb 20, 2022

Conversation

allenporter
Copy link
Collaborator

@allenporter allenporter commented Feb 20, 2022

Add a check for nil hlsMixer similar to the fix in commit 7beb35a

[14:51:47] INFO: Successfully send discovery information to Home Assistant (8083).
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xb94673]
goroutine 101 [running]:
main.(*MuxerHLS).WritePacket(0x0, 0xc0011c9780)
	/workspace/hlsMuxer.go:48 +0x33
main.(*StorageST).HlsMuxerWritePacket(0xc0000320f0, 0xc000038f3d, 0xd, 0x12af380, 0x1, 0xc0011c9780)
	/workspace/storageStreamChannel.go:366 +0x1f9
main.StreamServerRunStream(0xc000038f3d, 0xd, 0x12af380, 0x1, 0xc0000e0370, 0x0, 0x0, 0x0)
	/workspace/streamCore.go:173 +0xb09
main.StreamServerRunStreamDo(0xc000038f3d, 0xd, 0x12af380, 0x1)
	/workspace/streamCore.go:43 +0x4fa
created by main.(*StorageST).StreamEdit
	/workspace/storageStream.go:65 +0x5ff
[cont-finish.d] executing container finish scripts...

Issue #89

Add a check for nil hlsMixer similar to the fix in commit 7beb35a
@allenporter allenporter merged commit 6d3ab26 into deepch:master Feb 20, 2022
@allenporter allenporter deleted the hls-muxer-crash branch February 21, 2022 20:25
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