Segmentierungsfehler - C

Question

Warum kehrt der folgende Code mit einem Segmentierungsfehler zurück? Wenn ich Zeile 7 auskommentiere, verschwindet der Seg-Fehler.

int main(void){ 
     char *s; 
     int ln; 
     puts("Enter String"); 
     // scanf("%s", s); 
     gets(s); 
     ln = strlen(s); // remove this line to end seg fault 
     char *dyn_s = (char*) malloc (strlen(s)+1); //strlen(s) is used here as well but doesn't change outcome 
     dyn_s = s; 
     dyn_s[strlen(s)] = '\0'; 
     puts(dyn_s); 
     return 0; 
    } 

Prost!

Answer 1

s ist ein nicht initialisierter Zeiger; Sie schreiben an einen zufälligen Speicherort im Speicher. Dies ruft undefiniertes Verhalten auf.

Sie müssen etwas Speicher für s reservieren. Auch never use gets; Es gibt keine Möglichkeit, zu verhindern, dass es den zugewiesenen Speicher überläuft. Verwenden Sie stattdessen fgets.

Answer 2

katastrophal schlecht:

int main(void){ 
     char *s; 
     int ln; 
     puts("Enter String"); 
     // scanf("%s", s); 
     gets(s); 
     ln = strlen(s); // remove this line to end seg fault 
     char *dyn_s = (char*) malloc (strlen(s)+1); //strlen(s) is used here as well but doesn't change outcome 
     dyn_s = s; 
     dyn_s[strlen(s)] = '\0'; 
     puts(dyn_s); 
     return 0; 
    } 

Besser:

#include <stdio.h> 
#define BUF_SIZE 80 

int 
main(int argc, char *argv[]) 
{ 
     char s[BUF_SIZE]; 
     int ln; 
     puts("Enter String"); 
     // scanf("%s", s); 
     gets(s); 
     ln = strlen(s); // remove this line to end seg fault 
     char *dyn_s = (char*) malloc (strlen(s)+1); //strlen(s) is used here as well but doesn't change outcome 
     dyn_s = s; 
     dyn_s[strlen(s)] = '\0'; 
     puts(dyn_s); 
     return 0; 
    } 

Best:

#include <stdio.h> 
#define BUF_SIZE 80 

int 
main(int argc, char *argv[]) 
{ 
     char s[BUF_SIZE]; 
     int ln; 
     puts("Enter String"); 
     fgets(s, BUF_SIZE, stdin); // Use fgets (our "cin"): NEVER "gets()" 

     int ln = strlen(s); 
     char *dyn_s = (char*) malloc (ln+1); 
     strcpy (dyn_s, s); 
     puts(dyn_s); 
     return 0; 
    } 

Answer 3

Ihre scanf("%s", s); wird als Kommentar gekennzeichnet. Das bedeutet, dass s nicht initialisiert ist. Wenn diese Zeile ln = strlen(s); ausgeführt wird, erhalten Sie einen Seg-Fehler.

Es hilft immer, einen Zeiger auf NULL zu initialisieren, und dann vor dem Verwenden des Zeigers auf Null zu testen.

Answer 4

Noch besser

#include <stdio.h> 
int 
main(void) 
{ 
    char *line = NULL; 
    size_t count; 
    char *dup_line; 

    getline(&line,&count, stdin); 
    dup_line=strdup(line); 

    puts(dup_line); 

    free(dup_line); 
    free(line); 

    return 0; 
} 

Answer 5

char *s does not have some memory allocated . You need to allocate it manually in your case . You can do it as follows 
s = (char *)malloc(100) ; 

Dies würde nicht zu Segmentation fault Fehler führen, da Sie nicht mehr