diff options
author | George Joseph <gjoseph@digium.com> | 2016-06-28 08:00:32 -0600 |
---|---|---|
committer | George Joseph <gjoseph@digium.com> | 2016-06-29 09:01:51 -0500 |
commit | 4045e6d8ba6677f86483df28d902090fd5b471cd (patch) | |
tree | 04c0b5b0f5fead892baab98643c66c342541e434 /include | |
parent | 541f0386941d536059dd4ab9b1fecce547bda71c (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 4ea94f9bb..3873324b1 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; }; /*! |