tree cfbb42ba9c3f7cb4d5c27c531011ffbbeb8196d9
parent 536dff1bb8867b0c572b833faedf94f4fcc3b015
author Neels Hofmeyr <neels@hofmeyr.de> 1565294341 +0200
committer Neels Hofmeyr <neels@hofmeyr.de> 1566264311 +0200

mgcp_codec: split codec_free() off of codec_init()

Both are used only in the same .c file, so make them static.

Move codec_set() guts into codec_add(): codec_set is only called by codec_add.
If codec_set were left separate, it'd look like the codec_init() is a bug and
lacks a codec_free() first. When looking at the entire context in codec_add(),
it becomes obvious that codec_init() should be called, not codec_free(),
because it is populating a previously unused entry.

Preparation to fix a memleak in a conn's codec list.

Change-Id: I120cab0a352a1e7b31c8f9c720c47b2c291311d7