为 strncpy 创建包装器以插入终止 null

Question

我决定为 strncpy 制作一个包装器，因为我的源代码需要我进行大量字符串复制。 我想确保如果源等于或大于目标，则字符串终止。

这段代码将在生产中使用，所以我只是想看看使用这个包装器是否有任何潜在的危险。

我以前从未做过包装，所以我正在努力使其完美。

非常感谢您的任何建议，

/* Null terminate a string after coping */
char* strncpy_wrapper(char *dest, const char* source, 
                      const size_t dest_size, const size_t source_size)
{
    strncpy(dest, source, dest_size);
    /* 
     * Compare the different length, if source is greater
     * or equal to the destination terminate with a null.
     */
    if(source_size >= dest_size)
    {
        dest[dest_size - 1] = '\0';
    }

    return dest;
}

====编辑更新====

/* Null terminate a string after coping */
char* strncpy_wrapper(char *dest, const char* source, 
                      const size_t dest_size)
{
    strncpy(dest, source, dest_size);
    /* 
     * If the destination is greater than zero terminate with a null.
     */
    if(dest_size > 0)
    {
        dest[dest_size - 1] = '\0';
    }
    else
    {
        dest[0] = '\0'; /* Return empty string if the destination is zero length */
    }

    return dest;
}

Answer 1

您不需要检查源是否大于目标，只需始终将目标中的最后一个字符设置为“\0”即可。

Answer 2

在访问数组之前检查dest_size，否则你会遇到麻烦：

if (dest_size > 0) {
    dest[dest_size - 1] = '\0';
}

---

实际上，现在我想起来了，最好还是死掉：

if (dest_size == 0) {
    fputs(stderr, "strncpy(x,y,0)");
    exit(1);
}

否则，你会遇到与原来相同的问题strncpy()，如果 dest_size 为 0，则 dest 可能不会终止。

Answer 3

您应该研究半标准 BSD 函数 strlcpy() 和 strlcat()。

您应该考虑 ISO TR24731 函数（例如 strncpy_s()）是否合适并且在您需要时可用。

Answer 4

如果您对缓冲区的分配没有任何限制，则 strndup 可能更合适。

它将分配和复制最多 len 个字符，并始终以 NULL 终止。

Answer 5

如果 dest_size 是要复制的字符数，source_size 是源字符串中的字符数。
你应该尝试这个：

 size_t numchars = dest_size > source_size ? source_size : dest_size;
 strncpy(dest,source,numchars) ;
 dest[numchars] = 0 ;