My Cプログラムがクラッシュする

Question

空のファイルを生成するプログラムを作成しようとしています。プログラムを実行しようとすると、コンソールからの入力を受けた後にクラッシュします。

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

int create(char* filename) 
{ 
    char filext[10]; 
    printf("\nEnter File Extension :"); 
    fgets(filext); 
     FILE* fp; 
     fp = fopen(strcat(filename,strcat(".",filext)),"w"); 
     if(!fp) 
     { 
      return 0; 
     } 
     fclose(fp); 
     return 1; 

} 
int main(int argc , char* argv[]) 
{ 
    int f; 
    int i; 
    char buffer[33]; 
    if (argc == 3) 
    { 
     for(i = 0; i < atoi(argv[2]) ; i++) 
     { 
      f = create(strcat(argv[1],itoa(i,buffer,10))); 
      if(f==0) 
      { 
       printf("error in creating files . check uac!!!"); 
      } 
      else{ 
       printf("\nfile Created ...\n"); 
      } 
     } 
    } 
    else{ 
     printf("syntax Error"); 
    } 
    return 0; 
} 

私はこのプログラムを実行しようとすると、私は拡張子にプログラムがクラッシュを入力した後、次の出力

F:\selfcreatedtools\filegen>gcc gen.c 

F:\selfcreatedtools\filegen>a level 100 

Enter File Extension :php 

を取得します。 私はプログラミングの初心者です。

Answer 1

あなたの主な問題は、

strcat(filename, "."); 
strcat(filename, filext); 
fp = fopen(filename, "w"); 

を試してみてください
fp = fopen(strcat(filename,strcat(".",filext)),"w");

のstrcat(".",filext)一部にあり、関数定義ヘッダはSIZEが小さい値である int create(char filename[SIZE])を（行われた場合、それは良いかもしれませんあなたがstrcat()を使用しているので、ユーザ定義の楽しみの中の文字列を変更するために、の代わりにint create(char* filename)のサイズがfilenameになります） ction create()。文字列が他のものに割り当てられたメモリに侵入した場合、エラーの原因となる不正なメモリアクセスは望ましくありません。

同様の問題がBLUEPIXYコメントにソリューションを提供しているためJonathan Lefflerによって指摘されるようargv[1]で文字列を変更するstrcat()を用いてあります。