On Wed, May 24, 2017, at 11:13 AM, Ross Beer wrote:
<snip>
>
> Therefore I have added the following code to check for this:
>
>
> if (format1->codec == NULL || format2->codec == NULL) {
> return AST_FORMAT_CMP_NOT_EQUAL;
> }
>
> The question is, should 'codec' be NULL if 'format1' and 'format2' are
> not NULL? Is adding the above check, the correct fix?
A format can't be created and remain valid without a codec being present
on it. A format itself is a codec + extra data about it. Identifying how
it became NULL and why the format is no longer valid would uncover the
real fix.
--
Joshua Colp
Digium, Inc. | Senior Software Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - US
Check us out at: www.digium.com & www.asterisk.org
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
