kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-11-19 08:11:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

pbx.c: Pre-initialize timezone to avoid crash on destroy

Browse Source 
In ast_build_timing, initialize the timezone value to NULL
in order to avoid deferencing an uninitialized value later
when calling ast_destroy_timing.  The timezone value could
be uninitialized if ast_build_timing were to fail due to a
zero length time string.

(closes issue ASTERISK-22861)
Reported by: Sebastian Murray-Roberts
Review: https://reviewboard.asterisk.org/r/3134/
Patches:
     ast_build_timing-initialize-timezone.patch uploaded by coreyfarrell (license 5909)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@406241 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Scott Griepentrog
2014-01-22 22:16:47 +00:00
parent 1dd94a2e19
commit 16c2728634
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
main/pbx.c
View File

@@ -8239,6 +8239,8 @@ int ast_build_timing(struct ast_timing *i, const char *info_in)
char *info;
int j, num_fields, last_sep = -1;
i->timezone = NULL;
/* Check for empty just in case */
if (ast_strlen_zero(info_in)) {
return 0;
@@ -8258,8 +8260,6 @@ int ast_build_timing(struct ast_timing *i, const char *info_in)
/* save the timezone, if it is specified */
if (num_fields == 5) {
i->timezone = ast_strdup(info + last_sep + 1);
} else {
i->timezone = NULL;
}
/* Assume everything except time */

4
utils/extconf.c
View File

@@ -4439,6 +4439,8 @@ int ast_build_timing(struct ast_timing *i, const char *info_in)
char *info;
int j, num_fields, last_sep = -1;
i->timezone = NULL;
/* Check for empty just in case */
if (ast_strlen_zero(info_in)) {
return 0;
@@ -4458,8 +4460,6 @@ int ast_build_timing(struct ast_timing *i, const char *info_in)
/* save the timezone, if it is specified */
if (num_fields == 5) {
i->timezone = ast_strdup(info + last_sep + 1);
} else {
i->timezone = NULL;
}
/* Assume everything except time */