このコードにはバグがありますが、見つけられません

Question

私はインタビューの準備をしており、私が理解できなかった練習問題を見つけました。コードは次のとおりです。

#include <stdio.h> 

int main() 
{ 
    char *p,*q; 
    p=(char *)malloc(25); 
    q=(char *) malloc(25); 
    strcpy(p,"amazon"); 
    strcpy(q,"hyd"); 
    strcat(p,q); 
    printf("%s\n",p); 
} 

コンパイルして実行しましたが、うまくいきました。コードに問題があり、初期化されていない変数と関係があると思われます。出力はamazonhydでなければなりません。誰かが私に何が欠けているかを教えてもらえますか？

編集：これは練習のインタビューの質問であり、このコードで何が問題なのかを質問しています。

Answer 1

が含まれています。

malloc関数にはstdlib.hが必要ですが、strcpyとstrcatにはstring.hが必要です。これらの関数がなければ、これらの関数はintに返されます。

これは、の場合に特に問題です。sizeof(int) != sizeof(void *)の場合です。暗黙定義は実際の定義と一致しないため、undefined behaviorを呼び出します。定義されていない動作のための方法の1つは、プログラムが適切に動作するように見えることです。ただし、この動作は保証されません。

プログラムが適合するためには、次のものを先頭に追加してください。

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

また、このプログラムはメモリを割り当てますが、freeは割り当てません。 mallocへの各呼び出しには、対応するfreeの呼び出しが必要です。そうしないと、メモリリークが発生します。

メモリリークを修正するためにprintfを呼び出した後、以下を追加します。