c - 如何解释此调试错误

Question

作为一名受训黑客，我决定着手制作我自己的 string_reverse 函数，它接受一个字符串，为一个新字符串分配内存，并返回一个指向新字符串的指针，但我没有得到我想要的期望，因为这会返回段错误。

#include <stdio.h>
#include <stdlib.h>


char* string_reverse(char* string);

char* string_reverse(char* string) {
  int len = 0;
  for (int i = 0; *(string + i) != '\0'; ++i)
    len++;

  char* result = (char*)malloc(len * sizeof(char));
  if (result == NULL){
    puts("Pointer failure");
    exit(EXIT_FAILURE);
  }

  for (int i = 0; *(string + i) != '\0'; ++i)
    *(result + (len - i)) = *(string + i);

  return *result;
}

int main() {
  char* str= "Ni Hao!";
  char* result = string_reverse(str);

  printf("%s\n", result);
  free(result);
  return 0;
}

作为返回，我收到了这条调试信息:

Starting program: /home/tmo/string_reverse 

Program received signal SIGSEGV, Segmentation fault.
0xb7e5b3b3 in strlen () from /lib/i686/cmov/libc.so.6

我应该如何解释这个结果？

Answer 1

您的代码没有将空终止符添加到反转的字符串中。结果，printf 函数在尝试计算它的长度时崩溃了。

将 malloc 行更改为以下内容

char* result = (char*)malloc((len+1) * sizeof(char));

并且您需要将以下行添加到 string_reverse 函数的末尾，以确保字符串具有空终止符。

result[len] = '\0';

一些其他评论

• 不需要 sizeof(char)。 char 的大小是 C 标准定义的少数类型之一，其值为 1。
• 第一个循环可以用对 strlen 的简单调用代替

编辑

另外两个问题。实际执行字符复制的行似乎不正确。我相信它应该是 (len - i - 1)。否则，初始字符写入将发生在 (result + len) 处，这是空终止符的位置。

*(result + ((len - i) - 1)) = *(string + i);

此外，不要在返回时取消引用结果