tree 587277acc9d2abd0eedee3d1aa59d84a59f9d0e1
parent 5a6220f43bfad495cf57a4062d0b7d0341aff505
author Neels Hofmeyr <neels@hofmeyr.de> 1565294341 +0200
committer neels <nhofmeyr@sysmocom.de> 1566942797 +0000

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