2つの文字列を受け入れるC関数の文字列を返します

Question

私のCの新しさを許してください！引数として2つのchar配列を受け取り、いくつかのJSONを返す関数を作成しようとしています。ここに私のコード、コンパイルの警告が続きます。このプログラムは、実行時に単に重複します。

#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); 
} 

gccでコンパイルする：あなたはcharとしてのchar *を返すされている

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

。

あなたの関数の宣言は次のようになります。

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

また、それは、そのサイズ与え合うことができるよりもjsonに多くのものを入れているようです。それは以前に予約していなかった255、 - char*からcharへの変換に数が0との間のメモリのいくつかのバイトを読み取るために、図1Bにのでprintf(json)試行を切り捨てているため

セグメンテーション違反が起こります。

Answer 2

• get_jsonは、ポインタchar*、ないcharを返す必要があります。
• 割り当てようとする長さにsidを含めるのを忘れてしまったため、プログラムが範囲外アクセスを引き起こし、の定義されていない動作を呼び出します。
• このプログラムには害はありませんが、一般的にユーザーからの文字列をprintf()という形式の文字列にすることは危険です。

このお試しください：

#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); 
} 

または単純本：

#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); 
}