c言語でセグメンテーションエラーが出ます

Question

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

#define MAX_SIZE 10
#define L_COUNT 5


void func(char *c_buf,char **tag,char **last_char){
  int i;

 tag[0] =c_buf;
  for(i=0;i<L_COUNT;i++){
    if(fgets(tag[i],MAX_SIZE,stdin)== NULL){
       *tag[i]='\0';
       tag[i+1]=tag[i]+1;
      
       }

    else{
      *last_char=tag[i]+strlen(tag[i])-1;
      if((**last_char=='\n')){
	*last_char='\0';
	tag[i+1]=*last_char+1;
	//  printf("-------\n");
      }
      else{
	int get_data;
	do{
	  get_data=getchar();
	}while(!((get_data=='\n')||(get_data==EOF)));
	tag[i+1]=*last_char+2;
      }
    }
  }
  
}

void func2(char **tag){

  int i,j;
  char *swap_buf;

  for(i=0;i<L_COUNT;i++){
    printf("%d : %s\n",i,tag[i]);
  }

  for(i=0;i<(L_COUNT-1);i++){
    for(j=L_COUNT-1;j>i;j--){
      if(strncmp(tag[j-1],tag[j],MAX_SIZE-1)>0){
	swap_buf=tag[j];
	tag[j]=tag[j-1];
	tag[j-1]=swap_buf;
      }
    }
  }

  printf("\n\n");
  for(i=0;i<L_COUNT;i++){
    printf("%d:%s\n",i,tag[i]);
  }
}


int main(){
  int i;
  char c_buf[L_COUNT *MAX_SIZE];
  char *tag[L_COUNT +1], *last_char;
 
  func(c_buf,tag,&last_char);
  func2(tag);


}

5つの文字列を入力してその文字列をアルファベット順に並び替えるプログラムを書いているのですが、コンパイルはできたのですが実行をしようとするとセグメンテーションエラーが出てしまいます。
関数funcで文字列を入力して、関数func2で文字列を並び替えています。

実行する際

blue
red

と2つ目の文字列を入力した後にセグメンテーションエラーが起きます。他の文字列でも同じでした。
どこが悪いのかわからないので分かる方教えていただきたいです。

Answer 1

funcの引数 last_char に混乱が見られます。
名前の付け方もそうですが、main に戻ってきてから使わないのであれば、引数にしない方が良い気もします。

      if((**last_char=='\n')){
    *last_char='\0';
    tag[i+1]=*last_char+1;

少なくともここを見ると、最初に **last_char に文字を格納している想定の比較をしているにも関わらず、次は *last_char に文字を格納していて矛盾があります。
最後、tagの要素に入るのは、数値0から変換されたポインタNULLに1足したものなので、アドレス値としては不正でしょう。
※これがセグメンテーションフォールトの根本原因に思えます。

なお、「どこが原因か」はデバッガで追えるようにした方が良いと思います。
高度な使い方をしなくても、デバッグ情報を埋め込んでコンパイルし、デバッガから実行かければ、最悪どこで落ちたかの情報を出してくれるはずです。

Answer 2

func関数の

    *last_char='\0';
    tag[i+1]=*last_char+1;

この結果、どうなるでしょうか? tag[1+1] は 1 じゃないかと。

気が付いたところで。
(他も気になるが、追いにくい....)

Answer 3

mainで宣言しているtagが初期化されてない。

tagは何だと思っているのでしょうか？

Comments

[otn]

簡単に直すには、2次元配列にすればいいかと思います。

[angel_p_57]

> tagは何だと思っているのでしょうか？

funcで各要素にアドレス値を格納する想定なのだから初期化は要らないでしょう。

[otn]

tag[0] =c_buf;
を見落としてました。