mirror of
https://github.com/asterisk/asterisk.git
synced 2025-09-04 20:04:50 +00:00
Fix a couple of memory leaks with frame handling. Specifically,
ast_frame_free() needed to be called on the frame that came from the translator to signed linear. git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@99187 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
@@ -64,6 +64,7 @@ int ast_slinfactory_feed(struct ast_slinfactory *sf, struct ast_frame *f)
|
||||
ast_translator_free_path(sf->trans);
|
||||
sf->trans = NULL;
|
||||
}
|
||||
|
||||
if (!sf->trans) {
|
||||
if ((sf->trans = ast_translator_build_path(AST_FORMAT_SLINEAR, f->subclass)) == NULL) {
|
||||
ast_log(LOG_WARNING, "Cannot build a path from %s to slin\n", ast_getformatname(f->subclass));
|
||||
@@ -72,7 +73,15 @@ int ast_slinfactory_feed(struct ast_slinfactory *sf, struct ast_frame *f)
|
||||
sf->format = f->subclass;
|
||||
}
|
||||
}
|
||||
if (!(begin_frame = ast_translate(sf->trans, f, 0)) || !(duped_frame = ast_frdup(begin_frame)))
|
||||
|
||||
if (!(begin_frame = ast_translate(sf->trans, f, 0)))
|
||||
return 0;
|
||||
|
||||
duped_frame = ast_frdup(begin_frame);
|
||||
|
||||
ast_frfree(begin_frame);
|
||||
|
||||
if (!duped_frame)
|
||||
return 0;
|
||||
} else {
|
||||
if (!(duped_frame = ast_frdup(f)))
|
||||
|
Reference in New Issue
Block a user