From 826474a1a3707bb837bb1b1aa687d4981acd914d Mon Sep 17 00:00:00 2001 From: Vincent Sanders Date: Mon, 4 Mar 2019 17:31:26 +0000 Subject: use memcpy instead of strcpy to make the dom string copy intention explicit --- content/handlers/html/imagemap.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'content/handlers/html/imagemap.c') diff --git a/content/handlers/html/imagemap.c b/content/handlers/html/imagemap.c index d26ba5f4d..0c3576842 100644 --- a/content/handlers/html/imagemap.c +++ b/content/handlers/html/imagemap.c @@ -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; } -- cgit v1.2.3