C - realloc 错误：损坏的大小与 prev_size

Question

有人能解释一下这个错误吗？
我收到了错误，直到一时兴起，我改变了这一行：

char *tmp = realloc(str, sizeof(char)*length);
// to                                           added 1
char *tmp = realloc(str, sizeof(char) * length + 1);

我认为将sizeof(char) 乘以长度会重新分配size=sizeof(char)*length 的新内存区域。我不明白为什么添加 1 可以解决问题。

    void edit_print(char *inputStr, size_t space_size) {
      size_t ch_position = 0;
      size_t space_column_count = 0;
      size_t num_spaces_left = 0;
      while ((inputStr[ch_position] != '\0')) {
        if ((inputStr[ch_position] == '\t') && (space_size !=0)) {
          num_spaces_left = (space_size-(space_column_count % space_size));
          if (ch_position == 0 || !(num_spaces_left)) {
            for (size_t i=1; i <= space_size; i++) {
              putchar(' ');
              space_column_count++;
            }
            ch_position++;
          } else {
            for (size_t i=1; i <= num_spaces_left; i++) {
              putchar(' ');
              space_column_count++;
            }
            ch_position++;
          }
        } else {
          putchar(inputStr[ch_position++]);
          space_column_count++;
        }
      }
      printf("\n");
    }

        int main(int argc, char *argv[]) {
          size_t space_size_arg = 3; 
          int inputch;
          size_t length = 0;
          size_t size = 10;
          char *str = realloc(NULL, sizeof(char) * size);
          printf("Enter stuff\n");

          while ((inputch = getchar()) != EOF) {
            if (inputch == '\n') {
              str[length++] = '\0';

              //changed line below
              char *tmp = realloc(str, sizeof(char) * length + 1);

              if (tmp == NULL) {
                exit(0);
              } else {
                str = tmp;
              }
              edit_print(str, space_size_arg);
              length = 0;
            } else {
              str[length++] = inputch;
              if (length == size) {
                char *tmp = realloc(str, sizeof(char) * (size += 20));
                if (tmp == NULL) {
                  exit(0);
                } else {
                  str = tmp;
                }
              }
            }
          }
          free(str);
          return 0;
        }

编辑：我最初收到的错误消息是这篇文章标题中的错误消息。进行chux建议的更改后，错误为“realloc(): invalid next size: *hexnumber**”

Answer 1

size 需要在inputch == '\n' 时更新。

char *tmp = realloc(str, sizeof(char) * length + 1 /* or no +1 */); 可以缩小分配。这使得后来的if (length == size) 无效（真正的分配大小更小），因此str[length++] = inputch; 失去了内存访问保护。更新 size 以修复该漏洞。

+1 不需要 - 它只是隐藏了问题，因为 + 1 并没有缩小分配。

  char *tmp = realloc(str, sizeof(char) * length);
  if (tmp == NULL) {
    exit(0);
  } else {
    str = tmp;
  }
  size = length; // add

---

关于sizeof(char)* 代码。按目标类型的大小进行缩放的想法很好，但对于 char，它并不重要，因为它总是 1. @Lee Daniel Crocker

如果代码想反映目标的类型可能会改变，不要使用size(the_type)，使用sizeof(*the_pointer)。更易于编码、审查和维护。

// Don't even need to code the type `str` points to
tmp = realloc(str, sizeof *str * length);