diff options
author | George Joseph <gjoseph@digium.com> | 2016-06-28 08:00:32 -0600 |
---|---|---|
committer | George Joseph <gjoseph@digium.com> | 2016-06-28 13:38:28 -0600 |
commit | d07c8a050445cf48591105f1676c190f477a7af1 (patch) | |
tree | 1819065588c5f87fedc3be5a120805e0cf90b235 /include | |
parent | f3d236ca7fdf8b03106f1e50474ff6a79f3efb20 (diff) |
codecs: Fix ABI incompatibility created by adding format_name to ast_codec
Adding format_name even to the end of ast_codec caused issued with
binary codec modules because the pointer would be garbage in asterisk
when they registered. So, the ast_codec structure was reverted and an
internal_ast_codec structure was created just for use in codec.c. A new
internal-only API was also added (__ast_codec_register_with_format) so
that codec_builtin could register codecs with the format_name in a
separate parameter rather than in the ast_codec structure.
ASTERISK-26144 #close
Reported-by: Alexei Gradinari
Change-Id: I6df1b08f6a6ae089db23adfe1ebc8636330265ba
Diffstat (limited to 'include')
-rw-r--r-- | include/asterisk/codec.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/asterisk/codec.h b/include/asterisk/codec.h index fb2b7da38..28befec50 100644 --- a/include/asterisk/codec.h +++ b/include/asterisk/codec.h @@ -77,8 +77,6 @@ struct ast_codec { unsigned int smooth; /*! \brief The module that registered this codec */ struct ast_module *mod; - /*! \brief A format name for a default sane format using this codec */ - const char *format_name; }; /*! |