MT#55283 codec_add_raw_packet_dup: allocate enough space to support e…

…ncrpytion

using str_init_dup_str doesn't leave enough room for appending encryption
related pieces to the end of the packet when used for, eg, dtmf
injection

closes #1819

Change-Id: Iefae0e04b38f4a3eaaac32ed1ba70c7e3ee8e979
(cherry picked from commit ce66c7b)
(cherry picked from commit 4fd891f)

daemon/codec.c

@@ -1638,7 +1638,13 @@ void codec_add_raw_packet(struct media_packet *mp, unsigned int clockrate) {
 }
 static void codec_add_raw_packet_dup(struct media_packet *mp, unsigned int clockrate) {
 	struct codec_packet *p = g_slice_alloc0(sizeof(*p));
-	str_init_dup_str(&p->s, &mp->raw);
+	// don't just duplicate the string. need to ensure enough room
+	// if encryption is enabled on this stream
+	char *buf = g_malloc(mp->raw.len + 1 + RTP_BUFFER_TAIL_ROOM);
+	if (mp->raw.s && mp->raw.len)
+		memcpy(buf, mp->raw.s, mp->raw.len);
+	p->s.len = mp->raw.len;
+	p->s.s = buf;
 	p->free_func = free;
 	p->rtp = (struct rtp_header *) p->s.s;
 	codec_add_raw_packet_common(mp, clockrate, p);