C++ CGIスクリプト、フォームデータを読む

Question

私はthis tutorialを使ってC++ CGIスクリプトを作っています。私は、フォームのPOSTデータを読み取るしようとすると、しかし、それはコンパイルしていない：

char* lpszContentLength = getenv("CONTENT_LENGTH"); 
    char* lpszBuffer; 
    int nContentLength = atoi(lpszContentLength); 

    lpszBuffer = malloc(lpszContentLength+1); // allocate a buffer 
    memset(lpszBuffer, 0, lpszContentLength+1); // zero it out 

    fread(lpszBuffer,1,lpszContentLength,stdin); // get data 

ここでは、コンパイラの不満です：

LN 12が「lpszBuffer」で始まるものです

cgi.cpp: In function ‘int main()’: 
cgi.cpp:12: error: invalid conversion from ‘char*’ to ‘size_t’ 
cgi.cpp:12: error: initializing argument 1 of ‘void* malloc(size_t)’ 
cgi.cpp:12: error: invalid conversion from ‘void*’ to ‘char*’ 
cgi.cpp:13: error: ‘memset’ was not declared in this scope 
cgi.cpp:15: error: invalid conversion from ‘char*’ to ‘size_t’ 
cgi.cpp:15: error: initializing argument 3 of ‘size_t fread(void*, size_t, size_t, FILE*)’ 

。

私はC++の新機能ですので、これを修正する方法がわからないのですが、なぜこれが起こっているのでしょうか。多分古くなったコードかもしれません...私は、喜んでPOSTリクエストからデータを読み込むための他の解決策を受け入れるでしょう。

編集： 皆さんの修正に応じてコードを更新しました。

char* lpszContentLength = getenv("CONTENT_LENGTH"); 
    char* lpszBuffer; 
    int nContentLength = atoi(lpszContentLength); 

    lpszBuffer = (char*)malloc(nContentLength+1); // allocate a buffer 
    memset(lpszBuffer, 0, nContentLength+1); // zero it out 

    fread(lpszBuffer,1,nContentLength,stdin); // get data 

しかし、私はまだATOIからセグメンテーションフォールトを取得：

==23419== Invalid read of size 1 
==23419== at 0x498DA8C: ____strtol_l_internal (strtol_l.c:298) 
==23419== by 0x498D7EF: strtol (strtol.c:110) 
==23419== by 0x498AB60: atoi (atoi.c:28) 
==23419== by 0x8048899: main (in .../cgi.cpp.cgi) 
==23419== Address 0x0 is not stack'd, malloc'd or (recently) free'd 

は、ここでの問題は何ですか？ POSTが空白の場合にPOSTフォームの送信と関係があると仮定しています。

Answer 1

これはC++のタイプキャストの問題です。 lpszBufferはchar*ですが、mallocはvoid*を返します。だから、それをchar*にキャストする必要があります。また、真でない整数値としてchar*（lpszContentLength）を使用しようとしていることにも注意してください。これは他の関数でも更新する必要があります。これは先にatoi関数を使用して変換したものです。その値を使用してください。

のでラインは、ソースファイルの先頭に、あなたが必要#include <string.h>memsetを使用するために最後に

lpszBuffer = (char*)malloc(nContentLength+1); 

を、お読みください。

また、（スクリプトがかなりの時間実行されている場合は）良い習慣として、完了したらメモリをfreeに忘れないようにしてください。つまり、mallocで割り当てられたものは、もう使用しないときに呼び出されるfreeを持つ必要があります。あなたは、関数の最後で、関数全体を通じてlpszBufferを使用するのであれば、単純にfree(lpszBuffer);

Answer 2

error: invalid conversion from ‘char*’ to ‘size_t’

を行うこれは、この場合には、char*ある（おそらく表す文字列をlpszContentLengthを使用しようとによって引き起こされます数字）をsize_t（通常は長整数）として指定します。これを解決するには、まずlpszContentLengthを整数値に変換する必要があります。私はそれを処理する必要があるエラーを検出するので、strtol()をお勧めします。

チュートリアルを見ると、彼らは既にatoi()を使って変換しています。誤ってlpszContentLengthを使用しようとしたところでnContentLengthを使用してこのエラーを修正することもできます。彼らのコードは決してテストされてはならない。

atoi()はエラーを検出しないため、チュートリアルを完了すると、適切なエラーチェックを使用してstrtol()に変更することをおすすめします。

---

error: invalid conversion from ‘void*’ to ‘char*’

これは、malloc関数の戻り値に(char*)にキャストを欠落していることが原因です。 C++では、キャストが必要とされる：

lpszBuffer = (char*)malloc(convertedNumber+1); 

---

error: ‘memset’ was not declared in this scope

これはmemsetのためのヘッダを含めないことによって引き起こされます。 man page for memsetあなたが追加する必要がありますを示しています：それは整数型である必要がありますとき

#include <string.h> 

---

error: invalid conversion from ‘char*’ to ‘size_t’

error: initializing argument 3 of ‘size_t fread(void*, size_t, size_t, FILE*)’

これらは両方とも、文字の文字列であることlpszContentLengthによって引き起こされます。上記の最初のエラーを参照してください。