From a917bb8c8dd53ad581f780c08e4104f9d95cae36 Mon Sep 17 00:00:00 2001
From: Richard Fuchs <rfuchs@sipwise.com>
Date: Thu, 15 Jul 2021 12:09:01 -0400
Subject: [PATCH] TT#14008 use appropriate allocation for a=rtcp-fb

The `str` objects are owned by the SDP parser, so we need to duplicate
those as well as the string content itself.

closes #1303

Change-Id: I329893812b3d0a5ac7961e1895f7e0ef21f016fd
(cherry picked from commit 3f0020274af99f0a10640f27d12a52132d9433c9)
---
 daemon/codec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemon/codec.c b/daemon/codec.c
index 6f4e2e433..33d86ec84 100644
--- a/daemon/codec.c
+++ b/daemon/codec.c
@@ -3308,7 +3308,7 @@ static void __rtp_payload_type_dup(struct call *call, struct rtp_payload_type *p
 	call_str_cpy(call, &pt->codec_opts, &pt->codec_opts);
 	for (GList *l = pt->rtcp_fb.head; l; l = l->next) {
 		str *fb = l->data;
-		call_str_cpy(call, fb, fb);
+		l->data = call_str_dup(call, fb);
 	}
 }
 static struct rtp_payload_type *__rtp_payload_type_copy(const struct rtp_payload_type *pt) {