Verkettung von String und int ergibt Segmentierungsfehler in C

Question

Ich bin mir nicht sicher, was ich falsch mache. Ich versuche, hostname mit pid zu verknüpfen, um id zu erstellen.

char *generate_id(void) { 
    int ret; 
    char id[1048]; 
    char hostname[1024]; 
    pid_t pid = getpid(); 
    //hostname[1023] = '\0'; 

    if ((ret = gethostname(hostname,1024) < 0)) { 
     perror("gethostname"); 
     exit(EXIT_FAILURE); 
    } 
    sprintf(id, "%s%d", pid); 
    printf("hostname is %s\n", hostname); 
    printf("The process id is %d\n", pid); 
    printf("The unique id is %s", id); 

    return id; 
} 

EDIT:

aktualisiert Code nach ein paar Antworten zu lesen:

char *generate_id(void) { 
    int ret; 
    char hostname[1024]; 
    pid_t pid = getpid(); 
    //hostname[1023] = '\0'; 

    if ((ret = gethostname(hostname,1024) < 0)) { 
     perror("gethostname"); 
     exit(EXIT_FAILURE); 
    } 

    int size = snprintf(NULL, 0, "%s%d", hostname, pid); 
    char * id = malloc(size + 1); 

    printf("hostname is %s\n", hostname); 
    printf("The process id is %d\n", pid); 
    printf("The unique id is %s\n", id); 

    return id; 
} 

EDIT:

Arbeitscode:

char *generate_id(void) { 
    int ret; 
    char hostname[1024]; 
    pid_t pid = getpid(); 
    //hostname[1023] = '\0'; 

    if ((ret = gethostname(hostname,1024) < 0)) { 
     perror("gethostname"); 
     exit(EXIT_FAILURE); 
    } 

    int size = snprintf(NULL, 0, "%s%d", hostname, pid); 
    char * id = malloc(size + 1); 
    sprintf(id, "%s%d", hostname, pid); 
    printf("hostname is %s\n", hostname); 
    printf("The process id is %d\n", pid); 
    printf("The unique id is %s\n", id); 

    return id; 
} 

Answer 1

Problem mit dem Format-String:

sprintf(id, "%s%d", pid); 

Ihr Format-String hat zwei Formatierer (%s für einen String und %d für ein int), doch nur ein pid_t passieren. Du meinst wahrscheinlich:

sprintf(id, "%s%d", hostname, pid); 

oder

sprintf(id, "%d", pid); 

In Ihrem Code, der %s die pid als Zeiger interpretiert. Der Versuch, die Formatierung der Zeichenfolge zu dereferenzieren, verursacht den Segmentierungsfehler, da es sich um einen ungültigen Zeigerwert handelt.

Problem mit Ihrer Speicherverwaltung:

Aber dann gibt es auch undefiniertes Verhalten in Ihrem Code: Sie erklären id ein Stapel zugeordnete Array sein, aber Sie Rückkehr das Array (die in einen Zeiger zerfällt hier). Dies ist auch falsch und kann später zu einem Absturz führen.

Sie müssen wie diese id zu einem Haufen zugeordnete Array ändern:

char * id = malloc(1024); 

Der Anrufer Ihrer generate_id Funktion muss dann den Speicher free, wenn es fertig ist.

Es ist wahrscheinlich eine gute Idee, nur den benötigten Speicherplatz zuzuweisen. Sie können snprintf dafür wie folgt verwenden:

// Determine how much space the string needs. 
int size = snprintf(NULL, 0, "%d", pid); 
// Allocate the required space plus NULL termination. 
char * id = malloc(size + 1); 
// Actually print the string. 
sprintf(id, "%d", pid); 

Answer 2

sprintf(id, "%s%d", pid); 

Sie haben zwei Selektoren% s und% d, aber nur einen Parameter (pid). Sie müssen eine Zeichenfolge und eine Ganzzahl statt nur der Ganzzahl eingeben.

Answer 3

nicht sicher, wo Sie sind Speicherzugriffsfehler, aber Sie haben ein paar Probleme.

snprintf() ist viel sicherer und wird den Puffer id [] nicht überschreiben. sprintf könnte den Puffer überlaufen

sprintf (id, "% s% d", pid) ist schlecht wie oben erwähnt.

Rückgabe-ID ist schlecht, da es den Zeiger auf einen Wert auf dem Stapel zurückgibt. Sobald du zurückkommst, gehört der Stapel dir nicht mehr.