Strcpy Segmentation Fault C

Ich Lerne einige neue Dinge stecken, die auf ein einfaches strcpy Betrieb.
Ich verstehe nicht, warum zum ersten mal, wenn ich drucken funktioniert, aber zweites mal nicht.

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

int main()
{
    char *name;
    char *altname;

    name=(char *)malloc(60*sizeof(char));
    name="Hello World!";
    altname=name;
    printf("%s  \n", altname);
    altname=NULL;
    strcpy(altname,name);
    printf("%s  \n", altname);
    return 1;
}

Sie sind nicht die Zuweisung von mehr Speicher für altname/name.
wow, das ist ineffizient. sizeof(char) ist garantiert 1 byte, und wenn Sie gehen, um zu reservieren eine Feste Speichergröße, warum malloc()?

InformationsquelleAutor user3623498 | 2014-08-27

c strcpy

Müssen Sie Speicher für altname :

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

int main()
{
    char *name;
    char *altname;

    name=(char *)malloc(60*sizeof(char));
    name="Hello World!";
    altname=name;
    printf("%s  \n", altname);
    altname=NULL;
    //allocate memory, so strcpy has space to write on ;)
    altname=(char *)malloc(60*sizeof(char));
    strcpy(altname,name);
    printf("%s  \n", altname);
    return 1;
}

InformationsquelleAutor lpg

6

Fangen die Probleme hier:
```
name=(char *)malloc(60*sizeof(char));
name="Hello World!";
```
Sie ersetzt den Wert, der zurückgegeben wird, durch malloc mit einem string-literal.
1. Du Speicherleck (da kann man nicht wieder die Zeiger Rückgabewert malloc). Alle Anrufe zu malloc sind abgestimmt mit einem entsprechenden Aufruf free. Da Zeiger mit dem Wert Weg ist, die Gelegenheit zu nennen free mit diesem Zeiger Wert ist auch verschwunden.
2. Sie weiter schreiben Sie einen NULL-Zeiger, was zu undefiniertem Verhalten (was in deinem Fall produziert einen Speicherzugriffsfehler).
- Ich habe die änderung auf meine Antwort.
InformationsquelleAutor PaulMcKenzie
2

Ersten mal, Sie machen altname Punkt an der gleichen Stelle wie name. Das ist OK, weil name Punkte, um eine gültige char* (das erste element des "Hello World!" literal)
```
//both point to beginning of "Hello World!" literal
altname=name;
```
Zweiten mal, Sie versuchen, kopieren Sie die Daten wies auf die von name in den Ort zeigte auf, indem altname, die in dieser Phase Punkte zu NULL. So dass Sie versuchen, zu schreiben, zu NULL, was ist die Quelle der Fehler.

strncpy erfordert, dass die Ziel-Puffer beschreibbar sein, und groß genug, um kopieren Sie die Quell-Zeichenfolge-Daten in. Sie brauchen, um altname Punkt auf einen Puffer, der groß genug ist für den Inhalt der Zeichenfolge name Punkte zu.
```
altname = (char*)malloc(60*strlen(name)+1); //+1 for nul terminator
strcpy(altname, name);
```
Beachten Sie auch, dass wenn Sie name = "Hello World!" Sie Leck Sie den Speicher, den er ursprünglich hinwies. Sie müssen frei, die erste:
```
free(name);
name = "Hello World!";
```
- Nein, tatsächlich, er legt altname=NULL, und versuche, Sie zu kopieren gibt, also null-Zeiger-Ausnahme, würde ich sagen.
- Oops, ich habe die NULL. Behoben.
InformationsquelleAutor juanchopanza

Sie versuchen, einen Wert zuweisen zu altname hat keine Speicherplatz zum speichern. Ersten Speicher zu altname weisen

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

int main()
{
    char *name;
    char *altname;

    name=(char *)malloc(60*sizeof(char));
    name="Hello World!";
    altname=name;
    printf("%s  \n", altname);
    altname=NULL;
    altname=(char *)malloc(sizeof(name)); //allocate memory
    strcpy(altname,name);                 //Now assign 
    printf("%s  \n", altname);
    return 1;
}

InformationsquelleAutor Yogeesh H T

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.