diff options
| author | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-04-15 20:17:33 +0000 |
|---|---|---|
| committer | mmichelson <mmichelson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-04-15 20:17:33 +0000 |
| commit | 343a0e2502690fcbca0146159ec4369e2b53af5f (patch) | |
| tree | 17a4a1b7f8701ff511ef822aa0249ddd667fc961 /main | |
| parent | 63308097d4363328acdd935b255cf84bb105ecb8 (diff) | |
Merged revisions 188582 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r188582 | mmichelson | 2009-04-15 15:04:20 -0500 (Wed, 15 Apr 2009) | 7 lines
Update ast_readvideo_callback to match ast_readaudio_callback.
This fixes potential refcount errors that may occur on ast_filestreams.
AST-208
........
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@188585 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main')
| -rw-r--r-- | main/file.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/main/file.c b/main/file.c index ac47b2b51..0439d03a8 100644 --- a/main/file.c +++ b/main/file.c @@ -778,12 +778,17 @@ static enum fsread_res ast_readvideo_callback(struct ast_filestream *s) ast_set_flag(fr, AST_FRFLAG_FROM_FILESTREAM); ao2_ref(s, +1); } - if (!fr || ast_write(s->owner, fr)) { /* no stream or error, as above */ - if (fr) + if (!fr /* stream complete */ || ast_write(s->owner, fr) /* error writing */) { + if (fr) { ast_log(LOG_WARNING, "Failed to write frame\n"); + ast_frfree(fr); + } s->owner->vstreamid = -1; return FSREAD_FAILURE; } + if (fr) { + ast_frfree(fr); + } } if (whennext != s->lasttimeout) { |