C言語：自作関数に関して

Question

質問

char *args[256]があったとして、これを関数に渡し、これを返すような関数の宣言の仕方として次は正しいでしょうか？

// プロトタイプ宣言
char **fuction(char*[])

int main(){
   char *args[256];
   
   args = function(args);
}

char **funtion(char *args[]){
   int i;
   for(i = 0; ; ++i){
     if(strstr(args[i], "*") != NULL) break;
     if(args[i] == NULL) return args;
   }
   
   ( 省略 )
   return args;
}

現状

printfでargs[i]を出力しようとしたり、実行してみたりしましたが、セグメンテーション違反になります。
原因はここではないかもしれませんが。

argsの構造

argsは例えば次のような構造をとっています：
args[0] = ls
args[1] = *
args[2] = NULL

いずれにせよ、この関数の書き方が正しいかどうか、また、間違っている場合にはどのようにすればよいかご教授ください。

よろしくお願いいたします。

Answer 1

「関数の書き方」という点ではまぁそれでもいいかと思います(返り値argsにどういう意味があるかは疑問)。
しかし、変数の内容が関数の(というか処理の)前提を満たさないものになっています。

関数に関係なく、

int main(){
   char *args[256];
   //関数を展開
   for(int i = 0; ; ++i){
     if(strstr(args[i], "*") != NULL) break;
     if(args[i] == NULL) break ;
   }
   return 0;
}

としても、かなりの確率でメモリアクセスエラーが出るのではないでしょうか。
なぜそうなるのか、考えてみてください。(すでに他の回答でネタは提供されていますが)

teratailの「質問するときのヒント」にこんな一節があります。

最も良いのは、現象を再現するためのミニマムなプログラムを改めて作ることです。そうすれば、貴方自身が現象と問題をより良く理解することにもつながるからです。

Answer 2

何の問題もなさそうですが...

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

// プロトタイプ宣言
char **function(char*[]);

int main(){
   char *args[256];
   args[0] = "Hello";
   args[1] = "world";
   args[2] = "!";
   args[3] = "*";
   args[4] = NULL;
   char** p = args;
   p = function(p);
   p = function(p);
   p = function(p);
   return 0;
}

char **function(char *args[]){
   int i;
   for(i = 0; ; ++i){
     if(strstr(args[i], "*") != NULL) break;
     if(args[i] == NULL) return args;
     printf("[%s] ", args[i]);
   }
   printf("\n");
   return args;
}

Answer 3

char *args[256];

と定義していますが、ローカル変数はその中身にはでたらめな値が入っています

まあ、でたらめな値、ってのはわかりづらいので、全て０が入るとしましょう。
そうした場合、あなたのそのコードではどういう事が起こるのか理解できるでしょうか