Skip to content

Crash fix: Guard against null hlsMuxer#104

Merged
allenporter merged 1 commit into
deepch:masterfrom
allenporter:hls-muxer-crash
Feb 20, 2022
Merged

Crash fix: Guard against null hlsMuxer#104
allenporter merged 1 commit into
deepch:masterfrom
allenporter:hls-muxer-crash

Conversation

@allenporter
Copy link
Copy Markdown
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