c - 使用 gcc -O2 优化如何解决问题 : pointer pointing nothing and having integer value as -23 in it, 消失了？

Question

我的项目中有一段代码给我带来了一些问题。

char* reverseString (char* str){
    char* outputString = NULL; 
    /* Calculate length of the string */
    int len = strlen(str);
    outputString = (char *)malloc(len*(sizeof(char)));   
    int *p = (int*)malloc(sizeof(int));
    int x = 5;
    *p = 25;
    free(p);
    return outputString;
}

在以下测试用例中:

input char input1[]="a";     
char input2[]="";

调用两个输入

char* res = reverseString(input1);
char* res2 = reverseString(input2);

我在项目中执行了如下代码:

char str2[] = "a";
char* res = reverseString(str2);
if (strlen(res2)==0 || res==NULL)
    printf("hello");
printf("%s",res);       // No output came out
char str3[] = "";
char* res2 = reverseString(str3);
if (strlen(res2)==0 || !*res2)
    printf("hello");
printf("%s",res2);

free(res);
free(res2);

根本没有输出res 和 res2 没有指向任何东西，甚至没有指向 NULL。如果输入字符串的长度大于 2，此代码也有效我面临的问题是 outputString 什么都没有，实际上什么都没有。我尝试与 NULL/(void*) 比较我能想到的每一种可能的组合:

(str[0]=='\0' || str[0]=='0' || !str[0] || str[0]==(char)0 || *str ==   (char)0 ||(!*str) ||strlen(str)==0 )

它的 strlen(outputString) 结果是 1

我还看到outputString的地址在调用free(p)前后没有变化如果我注释掉 free(p) 那么一切正常，

然后我将该代码分成一个小程序，但问题没有重现。

我做到了

printf("%c %d",outputString[0], outputString[0]-'0');

没有打印出一个字符，只打印出一个整数-23。

然后我用 gcc -O2 编译了代码然后问题突然解决了。

还有其他事情解决了我的问题在下一次函数调用之前释放内存。像下面这样:

char str2[] = "a";
char* res = reverseString(str);
if (strlen(res)==0 || res==NULL)
    printf("hello");     // This gave the output hello
printf("%s",res);   
free(res);
char str3[] = "";
char* res2 = reverseString(str3);
if (strlen(res)==0 || !*res2)
    printf("hello");       // This gave the output hello
printf("%s",res2);

免费(res3)；

这很好用。

我尝试了很多以了解出了什么问题。有人可以告诉我问题是什么，出了什么问题以及优化的使用如何解决它。为什么我最后不能一次释放所有内存。因为我仍然有指向在该函数调用中分配的内存的指针。

另外，为了调试这个问题，我写了一个 MCVE，但它没有重现这个问题。它仅在项目内发生。因此没有必要提供 MCVE

在 MCVE 中与 NULL 的比较是有效的，但令人惊讶的是在我的项目中没有

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

char* reverseString (char* str){
char* outputString = NULL;
/* Calculate lenght of the string */
int len = strlen(str);
/* allocate memory of size equal to the length of the given string */
outputString = (char *)calloc(len*(sizeof(char)),0);

 int *p = (int*)malloc(sizeof(int));
 int x = 5;
 *p = 25;
 free(p);
 return outputString;
}
int main()
{

char str2[] = "a";
char* res2 = reverseString(str2);
if (strlen(res2)==0 || res2==NULL)
    printf("hello");
printf("%s",res2);   // No output as expected.
char str3[] = "";
char* res3 = reverseString(str3);
if (strlen(res3)==0 || !*res3)
    printf("hello");
printf("%s",res3);

free(res2);
free(res3);
}

我还修改了我的代码，以便在函数调用结束时释放 outputString: 免费(资源)然后免费调用下一个电话(res2)；

Answer 1

您需要将一些数据复制到输出缓冲区!现在内容可以是任何内容。

此外，您需要为字符串终止符分配空间。

所以:

outputString = malloc(len + 1);  // +1 to fit string terminator

并添加:

for (i=0; i<len; i++) {
    outputString[i] = str[len - i - 1];
} 
outputString[len] = '\0';

您的 MCVE 没有重现问题的原因是因为它正在使用 calloc，所以分配的缓冲区填充了 0。