sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

AST-2018-010: Fix length of buffer needed for SRV and NAPTR results 

When dn_expand was being called on SRV and NAPTR results, the
return value was being used to calculate the size of the buffer
needed to store the host names.  Since dn_expand returns the
length of the COMPRESSED name the buffer could be too short
to hold the EXPANDED name.  The expanded name is NULL terminated
so using strlen() is the correct way to determine the length
actually needed for the buffer.

ASTERISK-28127
Reported by: Jan Hoffmann

patches:
  patch.diff submitted by janhoffmann (license 6986)

Change-Id: I4d35d6c431c6c6836cb61d37b1378cc47f0b414d
This commit is contained in:
George Joseph 2018-10-25 09:25:58 -06:00
parent bd5dde4e83
commit 009d67cd28
2 changed files with 20 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
main/dns_naptr.c
View File

@ -393,6 +393,7 @@ struct ast_dns_record *dns_naptr_alloc(struct ast_dns_query *query, const char *
int replacement_size;
const char *end_of_record;
enum flags_result flags_res;
size_t naptr_len;
ptr = dns_find_record(data, size, query->result->answer, query->result->answer_size);
ast_assert(ptr != NULL);
@ -435,7 +436,14 @@ struct ast_dns_record *dns_naptr_alloc(struct ast_dns_query *query, const char *
return NULL;
}
replacement_size = dn_expand((unsigned char *)query->result->answer, (unsigned char *) end_of_record, (unsigned char *) ptr, replacement, sizeof(replacement) - 1);
/*
* The return value from dn_expand represents the size of the replacement
* in the buffer which MAY be compressed. Since the expanded replacement
* is NULL terminated, you can use strlen() to get the expanded size.
*/
replacement_size = dn_expand((unsigned char *)query->result->answer,
(unsigned char *) end_of_record, (unsigned char *) ptr,
replacement, sizeof(replacement) - 1);
if (replacement_size < 0) {
ast_log(LOG_ERROR, "Failed to expand domain name: %s\n", strerror(errno));
return NULL;
@ -475,7 +483,9 @@ struct ast_dns_record *dns_naptr_alloc(struct ast_dns_query *query, const char *
return NULL;
}
naptr = ast_calloc(1, sizeof(*naptr) + size + flags_size + 1 + services_size + 1 + regexp_size + 1 + replacement_size + 1);
naptr_len = sizeof(*naptr) + size + flags_size + 1 + services_size + 1
+ regexp_size + 1 + strlen(replacement) + 1;
naptr = ast_calloc(1, naptr_len);
if (!naptr) {
return NULL;
}

12
main/dns_srv.c
View File

@ -73,7 +73,13 @@ struct ast_dns_record *dns_srv_alloc(struct ast_dns_query *query, const char *da
return NULL;
}
host_size = dn_expand((unsigned char *)query->result->answer, (unsigned char *) end_of_record, (unsigned char *) ptr, host, sizeof(host) - 1);
/*
* The return value from dn_expand represents the size of the replacement
* in the buffer which MAY be compressed. Since the expanded replacement
* is NULL terminated, you can use strlen() to get the expanded size.
*/
host_size = dn_expand((unsigned char *)query->result->answer,
(unsigned char *) end_of_record, (unsigned char *) ptr, host, sizeof(host) - 1);
if (host_size < 0) {
ast_log(LOG_ERROR, "Failed to expand domain name: %s\n", strerror(errno));
return NULL;
@ -83,7 +89,7 @@ struct ast_dns_record *dns_srv_alloc(struct ast_dns_query *query, const char *da
return NULL;
}
srv = ast_calloc(1, sizeof(*srv) + size + host_size + 1);
srv = ast_calloc(1, sizeof(*srv) + size + strlen(host) + 1);
if (!srv) {
return NULL;
}
@ -94,8 +100,6 @@ struct ast_dns_record *dns_srv_alloc(struct ast_dns_query *query, const char *da
srv->host = srv->data + size;
strcpy((char *)srv->host, host); /* SAFE */
((char *)srv->host)[host_size] = '\0';
srv->generic.data_ptr = srv->data;
return (struct ast_dns_record *)srv;