Return String von C-Funktion akzeptiert zwei Strings

Question

Verzeihen Sie meine C newbness! Ich versuche, eine Funktion zu erstellen, die zwei Char-Arrays als Argumente akzeptiert und JSON zurückgibt. Hier ist mein Code, gefolgt von Kompilierungswarnungen. Das Programm segmentiert einfach bei der Ausführung.

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

char get_json(char *sid, char *obuf) 
{ 
     char *json; 
     json = malloc(strlen(obuf)+37); 
     strcpy(json, "{\"sessionline\":{\"sid\":\""); 
     strcat(json, sid); 
     strcat(json, "\",\"line\":\""); 
     strcat(json, obuf); 
     strcat(json, "\"}}"); 
     return json; 
} 

int main() 
{ 
     char *sid = "xyzxyzxyz"; 
     char *obuf = "asdfasdfasdfasdf"; 
     char *json = get_json(sid, obuf); 
     printf(json); 
} 

Wenn mit gcc kompilieren:

test.c: In function ‘get_json’: 
test.c:14:9: warning: return makes integer from pointer without a cast [enabled by default] 
     return json; 
     ^
test.c: In function ‘main’: 
test.c:21:22: warning: initialization makes pointer from integer without a cast [enabled by default] 
     char *json = get_json(sid, obuf); 
        ^
test.c:22:9: warning: format not a string literal and no format arguments [-Wformat-security] 
     printf(json); 
     ^

Answer 1

Sie sind ein char * als Zeichen zurück.

Ihre Funktionsdeklaration sollte wie folgt aussehen:

char* get_json(char *sid, char *obuf) 

Sie scheinen auch mehr Material in json zu setzen, als es aufgrund seiner Größe passen.

Die segfault geschieht, weil bei der Umwandlung von char* zu char die Nummer 1B abgeschnitten wird, so printf(json) versucht einige Byte Speicher zwischen 0 zu lesen - 255, die es vorher nicht behalten hat.

Answer 2

• get_json sollte einen Zeiger char* zurückkehren, nicht char.
• Sie haben vergessen, sid zu der Länge zuzuweisen, um zu reservieren, so dass Ihr Programm außerhalb des Bereichs Zugriff verursacht und undefined Verhalten aufrufen.
• Es gibt keinen Schaden in dieses Programm, aber gefährlicher Zeichenfolge aus Benutzern in das Formatstring von printf() ist in der Regel setzen.

Versuchen Sie folgendes:

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

char get_json(char *sid, char *obuf) 
{ 
     char *json; 
     json = malloc(strlen(sid)+strlen(obuf)+37); 
     if(json == NULL) return json; 
     strcpy(json, "{\"sessionline\":{\"sid\":\""); 
     strcat(json, sid); 
     strcat(json, "\",\"line\":\""); 
     strcat(json, obuf); 
     strcat(json, "\"}}"); 
     return json; 
} 

int main(void) 
{ 
     char *sid = "xyzxyzxyz"; 
     char *obuf = "asdfasdfasdfasdf"; 
     char *json = get_json(sid, obuf); 
     if (json != NULL) fputs(json, stdout); 
} 

oder einfacher diese:

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

char get_json(char *sid, char *obuf) 
{ 
     char *json; 
     json = malloc(strlen(sid)+strlen(obuf)+37); 
     if(json == NULL) return json; 
     sprintf(json, "{\"sessionline\":{\"sid\":\"" 
       "%s" 
       "\",\"line\":\"" 
       "%s" 
       "\"}}", sid, obuf); 
     return json; 
} 

int main(void) 
{ 
     char *sid = "xyzxyzxyz"; 
     char *obuf = "asdfasdfasdfasdf"; 
     char *json = get_json(sid, obuf); 
     if(json != NULL) fputs(json, stdout); 
}