Question

我有一个令人沮丧的问题，我无法找到答案。

我有这个功能：

// Append character to the end of a string
void str_AppendChar(char* s, const char* ch)
{
  // +2 because of 2x '\0'
  char* buff = malloc(strlen(s)+strlen(ch)+2);
  memset(buff, 0, sizeof(buff));

  // Copy the whole string in buff
  strcpy(buff, s);

  // Append ch at the end of buff
  int len = strlen(buff);
  strcpy(buff+len, ch);

  // Set end of the string
  *(buff+strlen(buff)-3) = '\0';

  strcpy(s, buff);

  free(buff);
}

由于某些原因，我的程序试图在结束时自由执行两次。

我使用AppendChar（）的代码是:(有点难看但跟我一起承担）

void it_GatherCmd(cmd_Details* lineptr[], char* cmd)
{
  // Used to count number of rows in lineptr
  int nlines;

  Detailptr p;
  char ch;
  char* word = (char*)malloc(sizeof(char)+256);
  memset(word, 0, sizeof(word));

  nlines = 0;
  while ((ch = *cmd++) != '\n')
  {
      if (ch != ' ' && ch != '\0' )
          str_AppendChar(word, &ch);
      else
      {
          int type = dict_CheckWord(word);

          if (type != -1)
          {
              p = it_CopyInfo(word, type);
              lineptr[nlines++] = p;
          }
          memset(word, 0, sizeof(word));
      }
   }
   //EDIT*
   free(word);
}

我的主要人物：

int main()
{
   cmd_Details* arrCmd[MAXLINES];
   char* str = "just some string";

   it_GatherCmd(arrCmd, str);

   printf("%s", str);
   return 0;
}

AppendChar（）工作没有问题，直到我创建了it_GetCharCmd（）并在那里使用它。我已经花了大约3个小时，我找不到问题。在互联网上进行了一些搜索，但我发现的事情与我的问题并不完全相关。

Answer 1

此代码存在一些问题。

首先，如果str_AppendChar实际上附加了一个字符，正如其名称所暗示的那样，为什么要给它一个const char*来暗示一个C字符串？这里有一个零增益传递指针而不是实际对象，就像某些结构的情况一样;实际上你还需要将4个字节压入堆栈。

其次，正如我在评论中指出的那样，问题是你没有正确地初始化分配的缓冲区 - sizeof(buff)返回良好，buff的大小，buff是{{1这很可能是4.虽然只是简单地将char*更改为sizeof(buff)，这实际分配了多少内存来解决问题（因为strlen(s)+strlen(ch)+2可能比你更多实际已分配，你写的是那段记忆），我建议简化这样的功能：

sizeof(buff)

请注意，此代码仍然不好;它愉快地假设s足够大以容纳一个额外的角色，但情况并非如此。

另外关于你的it_gatherCmd函数;它应该采用// Append character to the end of a string void str_AppendChar(char* s, char ch) { size_t sLen = strlen(s); char* buff = (char*)malloc(sLen + 2); // 1 for the appended char, 1 for \0 //memset(buff, 0, sLen + 2); //not necessary, we'll overwrite the memory anyway // Copy the whole string in buff strcpy(buff, s); // append our char and null-terminate buff[sLen] = ch; buff[sLen + 1] = '\0'; strcpy(s, buff); free(buff); }，因为它不会以任何方式修改它（实际上，你调用它的方式，你给它一个const char *;修改一个字符串literal是未定义的行为，在Windows上，您可能因违反页面权限而崩溃。

Answer 2

据我所见，您在扫描命令时连续构建一个字符串。在附加char时，绝对不需要复制字符串以追加两次。你实际做的是：

void str_AppendChar(char* s, char ch)
{
    int len = strlen(buff);

    s[len++] = ch;
    s[loen] = '\0';
}

请注意，每次都会使用strlen确定字符串长度，它将遍历整个字符串。更糟糕的是，您没有关于最大可用缓冲区大小的任何信息，因此尽管您分配，复制和释放所有内容，原始字符串可能会溢出。

而不是

str_AppendChar(word, &ch);

在自动内存中使用本地缓冲区：

char word[20];
int wlen = 0;

并且像这样追加：

if (wlen + 1 < sizeof(word)) word[wlen++] = *cmd;

这将使单词缓冲区无法终止，因此在使用之前，请将其附加到：

word[wlen] = '\0';
printf("next word: '%s'\n", word);

（或者你可以确保字符串在任何时候都是空终止的。）

当您为一个新单词重置缓冲区时，不需要memset整个缓冲区;只需将wlen重置为零。

双重free（）发生的地方不应该

2 个答案: