kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-13 16:21:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_rtp_asterisk.c: Fix off-nominal crash potential.

Browse Source 
ASTERISK-25296
Reported by: Richard Mudgett

Change-Id: I08549fb7c3ab40a559f41a3940f3732a4059b55b
This commit is contained in:
Richard Mudgett
2015-07-29 17:00:05 -05:00
parent 1519eb44a7
commit 84262749d2
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
res/res_rtp_asterisk.c
View File

@@ -4504,6 +4504,10 @@ static struct ast_frame *ast_rtp_read(struct ast_rtp_instance *instance, int rtc
} }
payload = ast_rtp_codecs_get_payload(ast_rtp_instance_get_codecs(instance), payloadtype); payload = ast_rtp_codecs_get_payload(ast_rtp_instance_get_codecs(instance), payloadtype);
if (!payload) {
/* Unknown payload type. */
return AST_LIST_FIRST(&frames) ? AST_LIST_FIRST(&frames) : &ast_null_frame;
}
/* If the payload is not actually an Asterisk one but a special one pass it off to the respective handler */ /* If the payload is not actually an Asterisk one but a special one pass it off to the respective handler */
if (!payload->asterisk_format) { if (!payload->asterisk_format) {
@@ -4530,10 +4534,7 @@ static struct ast_frame *ast_rtp_read(struct ast_rtp_instance *instance, int rtc
/* Even if no frame was returned by one of the above methods, /* Even if no frame was returned by one of the above methods,
* we may have a frame to return in our frame list * we may have a frame to return in our frame list
*/ */
if (!AST_LIST_EMPTY(&frames)) { return AST_LIST_FIRST(&frames) ? AST_LIST_FIRST(&frames) : &ast_null_frame;
return AST_LIST_FIRST(&frames);
}
return &ast_null_frame;
} }
ao2_replace(rtp->lastrxformat, payload->format); ao2_replace(rtp->lastrxformat, payload->format);