二进制到十进制分段错误

Question

我使用指针代替数组，我知道与数组不同，指针需要被释放。为什么使用指针代替数组会给我一个分段内存错误。

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

void bin(void){

char *input;
int choice;
int x = 0;


    printf("Enter Decimal Code:\n");
    scanf("%s",&input);

    int leng = strlen(input);



    for(int i = 0; i <= leng ; ++i){

        if(input[i] == '1'){
            x += pow(2,i);


        }
        else if(input[i] == '0'){
            input[i] = 0;


        }
        free(input);

    }
        printf("Binary-Dec: %d\n",x);


}

int main()
{
bin();

}

Answer 1

在你的代码中

 scanf("%s",&input);

是有问题的部分。如我所见，有两个问题。

• 首先，input 是指针，&input 是指针的地址。
• 其次，即使去掉&，由于input没有分配内存，还是会导致undefined behavior。

你应该这样做

 char input[32] = {0};
 scanf("%31s", input);   //limit the input buffer to prevent overflow

此外，建议您检查scanf() 的返回值是否成功，以确保正确输入。

---

FWIW，用于声明

[...]我知道需要释放指针[..]

正确，但仅当它们通过malloc() 或函数系列动态分配内存时。考虑以下情况，

char buf[32] = {0};
char *input = buf;
. . . . 

在这种情况下，您不需要以任何方式输入free()，因为内存不是动态分配的。更多信息，您可以参考this answer。

Answer 2

因为你的指针是unitialized，所以undefined behaviour会出现，你的代码可能会也可能不会。

更多信息请参见this。

Answer 3

我所要做的就是替换

   scanf("%s",&input);

与

   scanf("%ms",&input);

感谢 BLUEPIXY