In this case, using strdup will provide a cleaner code
(and maybe a little bit faster too).
Signed-off-by: Dotan Barak <dotanb@dev.mellanox.co.il>
Signed-off-by: Sean Hefty <sean.hefty@intel.com>
if (!addr)
return ERR(ENOMEM);
- canonname = ai->ai_canonname ? malloc(strlen(ai->ai_canonname) + 1) : NULL;
- if (canonname)
- strcpy(canonname, ai->ai_canonname);
+ canonname = ai->ai_canonname ? strdup(ai->ai_canonname) : NULL;
memcpy(addr, ai->ai_addr, ai->ai_addrlen);
if (ai->ai_flags & RAI_PASSIVE) {