发表新帖

strncpy的一些问题

swoole的代码中好多处地方使用了strncpy. strncpy函数存在两个缺陷

1：浪费CPU， 当src_size  <  dst_size的时候， 会复制（dst_size - src_size）个0， 实际上1个就够了

2，当src_size >= dst_size的时候，dst_size后来后面不会补0， 这个很容易导致bug.
我发现swoole中strncpy目前导致了两个问题

1，bug
{{{
    bzero(ls->host, SW_HOST_MAXSIZE);
    strncpy(ls->host, host, SW_HOST_MAXSIZE);
}}}
{{{
int swSocket_bind(int sock, int type, char *host, int port)
{
    // ............
    // 这里的host不一定以0结尾的， 有可能导致core dump.
     strcpy(addr_un.sun_path, host);
     // ............
}
}}}

2 ， 导致代码非常啰嗦

{{{
int get_spilt(char *pszSrc, char *pszSplit, char *pszDest, int nDestLen)
{
        iPos = strlen(pszSrc);
        iPos = iPos > (nDestLen - 1) ? (nDestLen - 1): iPos;
        strncpy(pszDest, pszSrc, iPos);
        pszDest[iPos] = '\0';
        // 这里可以用一句代码替代
        strlcpy(pszDest, pszSrc, nDestLen);
}
}}}

strlcpy 虽然不是标准函数， 但是目前在linux内核，chrome等大型工程都是事实上的标配。
可以引入一个函数来有效解决这个问题。
{{{
size_t swoole_strlcpy(char* dst, const char* src, size_t dst_size)
{
  for (size_t i = 0; i < dst_size; ++i)
  {
    if ((dst[i] = src[i]) == 0)
      return i;
  }

if (dst_size != 0)
    dst[dst_size - 1] = 0;

while (src[dst_size]) ++dst_size;
  return dst_size;
}
}}}

现在的问题是， 我想提交代码
1， github pull request, 目前还没有人给swoole提交过。
2， 直接提交没有权限。

我的github是:  https://github.com/cloudpeak

发布于8年前 · 0 次浏览 · 来自提问

MR.Liu

swoole的代码中好多处地方使用了strncpy. strncpy函数存在两个缺陷

1：浪费CPU， 当src_size  <  dst_size的时候， 会复制（dst_size - src_size）个0， 实际上1个就够了

2，当src_size >= dst_size的时候，dst_size后来后面不会补0， 这个很容易导致bug.
我发现swoole中strncpy目前导致了两个问题

2 ， 导致代码非常啰嗦

if (dst_size != 0)
    dst[dst_size - 1] = 0;

while (src[dst_size]) ++dst_size;
  return dst_size;
}
}}}

现在的问题是， 我想提交代码
1， github pull request, 目前还没有人给swoole提交过。
2， 直接提交没有权限。

我的github是:  https://github.com/cloudpeak

登录后参与评论

2018-04-02

tracyp

2018-04-02

Peter

2018-04-02

sky

2018-04-02

张

张雷

2018-04-03

LOOP

2018-04-04

zjh

2018-04-04

zlg

2018-04-04

聚

聚米宋