From 09ecd96ef2fcfc17f86da15e5a19829e5cf9bf1f Mon Sep 17 00:00:00 2001 From: s3lph <5564491+s3lph@users.noreply.github.com> Date: Thu, 3 Dec 2020 21:15:57 +0100 Subject: [PATCH] Add nil checks to mumble text message handling --- bridge/mumble/handlers.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/bridge/mumble/handlers.go b/bridge/mumble/handlers.go index a6845955..40272633 100644 --- a/bridge/mumble/handlers.go +++ b/bridge/mumble/handlers.go @@ -19,6 +19,12 @@ func (b *Bmumble) handleTextMessage(event *gumble.TextMessageEvent) { if event.TextMessage.Sender != nil { sender = event.TextMessage.Sender.Name } + // If the text message is received before receiving a ServerSync + // and UserState, Client.Self or Self.Channel are nil + if event.Client.Self == nil || event.Client.Self.Channel == nil { + b.Log.Warn("Connection bootstrap not finished, discarding text message") + return + } // Convert Mumble HTML messages to markdown parts, err := b.convertHTMLtoMarkdown(event.TextMessage.Message) if err != nil {