[asterisk-dev] [Code Review] 4149: main/file.c: fix possible extra ast_module_unref to format modules

Corey Farrell reviewboard at asterisk.org
Tue Nov 4 17:11:14 CST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4149/
-----------------------------------------------------------

Review request for Asterisk Developers.


Bugs: ASTERISK-24492
    https://issues.asterisk.org/jira/browse/ASTERISK-24492


Repository: Asterisk


Description
-------

fn_wrapper only adds a reference to the format's module if the file was able to be opened.  If not this causes an unmatched ast_module_unref in filestream_destructor.


Diffs
-----

  /branches/11/main/file.c 427255 

Diff: https://reviewboard.asterisk.org/r/4149/diff/


Testing
-------

Verified the issue and fix with tests/apps/voicemail/play_message + r4141.


Thanks,

Corey Farrell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141104/982abd9b/attachment.html>


More information about the asterisk-dev mailing list