この単純なCオーバーフローを修正するにはどうすればよいですか？

Question

私はこれに非常に新しいです。私は誰かがこれをアプローチする最良の方法にいくつかの洞察を提供することができればライン12 + 19

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

#define BUF_SIZE (1024) 

int main(int argc, char* argv[]) { 
    char* inBuf; 
    char* outBuf; 
    char* fmt = "the winner is: %s"; 

    inBuf = (char*) malloc(BUF_SIZE); 
    if (inBuf == NULL) { 
    return -1; 
    } 
    read(0, inBuf, BUF_SIZE); 
    outBuf = (char*) malloc(BUF_SIZE); 
    if (outBuf == NULL) { 
    return -1; 
    } 
    sprintf(outBuf, fmt, inBuf); 
    fprintf(stdout, "%s\n", outBuf); 
    fprintf(stderr, "%s\n", outBuf); 
    free(inBuf); 
    free(outBuf); 
} 

でピックアップされたバッファオーバーフローを除去するために、このコードを再書き込みしようとしていますか？大変ありがとうございます。

Answer 1

read関数を使用してユーザー入力を読み取るため、文字列ではなく生のバイトを読み取ることになります。だからそれは何を読み込んだNULL終了バイトを含まないので、mallocによって返されたバッファが初期化されていないので、nullで終わる文字列を持っていません。

mallocの代わりにcallocを使用します。これは、すべてゼロに初期化されたバッファを返します。

inBuf = calloc(BUF_SIZE + 1, 1); 

これは、終端ヌル文字のための余分なバイトを残すことに注意してください。

また、エラーのreadの戻り値をチェックしなければならない、とあなたはmalloc/calloc/reallocの戻り値をキャストshoudn't。

出力バッファサイズも小さすぎます。少なくとも入力文字列と書式文字列のサイズ以上でなければなりません。

outBuf = (char*) malloc(BUF_SIZE + 1 + strlen(fmt));