eukara
/
WebSurf
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

use memcpy instead of strcpy to make the dom string copy intention explicit

This commit is contained in:
Vincent Sanders 2019-03-04 17:31:26 +00:00
parent 13a7004e6b
commit 826474a1a3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
content/handlers/html/imagemap.c
View File

@ -376,14 +376,14 @@ imagemap_addtolist(const struct html_content *c,
}
if (target != NULL) {
/* Copy target into the map */
/* Copy target dom string into the map data */
new_map->target = malloc(dom_string_byte_length(target) + 1);
if (new_map->target == NULL)
goto bad_out;
strncpy(new_map->target,
dom_string_data(target),
dom_string_byte_length(target));
memcpy(new_map->target,
dom_string_data(target),
dom_string_byte_length(target));
new_map->target[dom_string_byte_length(target)] = 0;
}