diff options
| author | Joshua Colp <jcolp@digium.com> | 2015-11-12 08:24:06 -0400 |
|---|---|---|
| committer | Joshua Colp <jcolp@digium.com> | 2015-11-12 09:46:54 -0400 |
| commit | 4e5bf12b33de8db9c53f571e4c4d5cb094a0d008 (patch) | |
| tree | ccd3c2dcfdac69b034aa0fab01f2176347726c94 /tests | |
| parent | b818d70533916aa80af7607f225e0b1e98f41648 (diff) | |
format_cap: Don't append the 'none' format when appending all.
When appending all formats of a type all the codecs are iterated
and added. This operation was incorrectly adding the ast_format_none
format which is special in that it is supposed to be used when no
format is present. It shouldn't be appended.
ASTERISK-25535
Change-Id: I7b00f3bdf4a5f3022e483d6ece602b1e8b12827c
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test_format_cap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/test_format_cap.c b/tests/test_format_cap.c index e1ae553c7..ed5427fa2 100644 --- a/tests/test_format_cap.c +++ b/tests/test_format_cap.c @@ -245,7 +245,7 @@ AST_TEST_DEFINE(format_cap_append_all_unknown) } else if (!ast_format_cap_has_type(caps, AST_MEDIA_TYPE_VIDEO)) { ast_test_status_update(test, "Added all media formats but no video formats exist when they should\n"); return AST_TEST_FAIL; - } else if ((ast_format_cap_count(caps) + 1) != ast_codec_get_max()) { + } else if ((ast_format_cap_count(caps) + 1) != (ast_codec_get_max() - 1)) { ast_test_status_update(test, "The number of formats in the capabilities structure does not match known number\n"); return AST_TEST_FAIL; } |