Reason is optional

pion · Aug 16, 2024 · 995637f · 995637f
1 parent 13319e0
commit 995637f
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/goodbye.go b/goodbye.go
@@ -133,7 +133,11 @@ func (g *Goodbye) Header() Header {
 // MarshalSize returns the size of the packet once marshaled
 func (g *Goodbye) MarshalSize() int {
 	srcsLength := len(g.Sources) * ssrcLength
-	reasonLength := len(g.Reason) + 1
+	// reason is optional
+	reasonLength := len(g.Reason)
+	if reasonLength > 0 {
+		reasonLength++
+	}
 
 	l := headerLength + srcsLength + reasonLength