As of the conversion to safe_strcpy() years ago there has been no need anymore to use snprintf() to prevent storing a not-nul-terminated string.
Signed-off-by: Jan Beulich <jbeul...@suse.com> --- a/xen/common/rangeset.c +++ b/xen/common/rangeset.c @@ -436,14 +436,7 @@ struct rangeset *rangeset_new( BUG_ON(flags & ~RANGESETF_prettyprint_hex); r->flags = flags; - if ( name != NULL ) - { - safe_strcpy(r->name, name); - } - else - { - snprintf(r->name, sizeof(r->name), "(no name)"); - } + safe_strcpy(r->name, name ?: "(no name)"); if ( (r->domain = d) != NULL ) {