kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-02 19:16:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

func_frame_drop: Fix typo referencing wrong buffer

Browse Source 
In order to get around the issue of certain frames
having names that could overlap, func_frame_drop
surrounds names with commas for the purposes of
comparison.

The buffer is allocated and printed to properly,
but the original buffer is used for comparison.
In most cases, this wouldn't have had any effect,
but that was not the intention behind the buffer.
This updates the code to reference the modified
buffer instead.

ASTERISK-29854 #close

Change-Id: I430b52e14e712d0e62a23aa3b5644fe958b684a7
This commit is contained in:
Naveen Albert
2022-01-11 18:33:34 +00:00
committed by Friendly Automation
parent 9c9083b45a
commit 5df5a70d37
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
funcs/func_frame_drop.c
View File

@@ -227,13 +227,13 @@ static int frame_drop_helper(struct ast_channel *chan, const char *cmd, char *da
buffer = ast_malloc(sizeof(value) + 3); /* leading and trailing comma and null terminator */
snprintf(buffer, sizeof(value) + 2, ",%s,", value);
for (i = 0; i < ARRAY_LEN(frametype2str); i++) {
if (strcasestr(value, frametype2str[i].str)) {
if (strcasestr(buffer, frametype2str[i].str)) {
framedata->values[i] = 1;
}
}
for (i = 0; i < ARRAY_LEN(controlframetype2str); i++) {
if (strcasestr(value, controlframetype2str[i].str)) {
if (strcasestr(buffer, controlframetype2str[i].str)) {
framedata->controlvalues[i] = 1;
}
}