Include XR.SenderSSRC in its DestinationSSRC

pion · Mar 1, 2024 · a3dcc3f · a3dcc3f
1 parent f260c33
commit a3dcc3f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/extended_report.go b/extended_report.go
@@ -646,7 +646,8 @@ func (x *ExtendedReport) Unmarshal(b []byte) error {
 
 // DestinationSSRC returns an array of SSRC values that this packet refers to.
 func (x *ExtendedReport) DestinationSSRC() []uint32 {
-	ssrc := make([]uint32, 0)
+	ssrc := make([]uint32, 0, len(x.Reports)+1)
+	ssrc = append(ssrc, x.SenderSSRC)
 	for _, p := range x.Reports {
 		ssrc = append(ssrc, p.DestinationSSRC()...)
 	}

diff --git a/extended_report_test.go b/extended_report_test.go
@@ -272,4 +272,14 @@ func TestDecode(t *testing.T) {
 	if p.String() != pktStringer.String() {
 		t.Errorf("(string compare) Decoded packet does not match expected packet")
 	}
+
+	var includeSenderSSRC bool
+	for _, ssrc := range p.DestinationSSRC() {
+		if ssrc == p.SenderSSRC {
+			includeSenderSSRC = true
+		}
+	}
+	if !includeSenderSSRC {
+		t.Errorf("DestinationSSRC does not include the SenderSSRC")
+	}
 }