From: Phil Sutter
> Sent: 01 September 2017 17:53
> By making use of strncpy(), both implementations are really simple so
> there is no need to add libbsd as additional dependency.
> 
...
> +
> +size_t strlcpy(char *dst, const char *src, size_t size)
> +{
> +     if (size) {
> +             strncpy(dst, src, size - 1);
> +             dst[size - 1] = '\0';
> +     }
> +     return strlen(src);
> +}

Except that isn't really strlcpy().
Better would be:
        len = strlen(src) + 1;
        if (len <= size)
                memcpy(dst, src, len);
        else if (size) {
                dst[size - 1] = 0;
                memcpy(dst, src, size - 1);
        }
        return len - 1;

WTF strlcpy() has that return value I don't know.

        David


Reply via email to