c言語 pingコマンドを実行するプログラムについて

Question

前提・実現したいこと

c言語で./a.out arg1 arg2...とすればping -c 3 arg1, ping -c 3 arg2,...と実行するプログラムを作っています。
第二引数以降の結果も表示させたいのですが、第一引数の結果しか表示させられません。どう改良すべきでしょうか。

実行結果

./a.out red01 red02
PING red01 (xxx.xx.xx.x) 56(84) bytes of data.
64 bytes from red01 (172.21.93.1): icmp_seq=1 ttl=64 time=0.851 ms
64 bytes from red01 (172.21.93.1): icmp_seq=2 ttl=64 time=1.08 ms
64 bytes from red01 (172.21.93.1): icmp_seq=3 ttl=64 time=0.944 ms

--- red01 ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2002ms
rtt min/avg/max/mdev = 0.851/0.959/1.084/0.102 ms

該当のソースコード

#include <stdio.h>
#include <unistd.h>

int main(int argc,char *argv[]){
 int i;
 argv[argc]=NULL;
 for(i=1;argv[argc]!=NULL; i++){
    execl("/bin/ping", "ping", "-c", "3", argv[i]);
  }
}

Comments

[jimbe]

execl はプロセス自体をそのコマンドに変えてしまうと思いますが、どこまでご理解されていますでしょうか。

[dodox86]

質問の内容とは直接関係ないですが、

> argv[argc]=NULL;

こんなことしちゃダメでしょう、と。何か参考にされたコードがあったのでしょうか。

[cgen]

>jimbe様
ping -c 3　を実行するプログラムを作りたくてネットにあったコードを参照して書いてみました、、

[cgen]

>dodox86様
友達の書いたコードを参照しました、、。なぜダメなのか教えていただけないでしょうか。

[jimbe]

> ネットにあったコードを参照
でしたら、 execl についてもお調べになることをお勧めします。

[cgen]

>ありがとうございます！！
最後の引数にNULLが必要なのですね！また複数のプログラムを実行したい場合はforkシステムコールが必要だとわかりました。もう一度考えてみます。

Answer 1

argv[argc]=NULL;

これだめです。
そもそも、argvのサイズはargcで与えられてるんでそれ使いましょうよ

for(i=1;argv[argc]!=NULL; i++){

これも間違ってますぜ

Comments

[cgen]

ありがとうございます！！
for(i=1;i<argc;i++)
にすれば大丈夫でしょうか。またなぜ上の二行がだめなのか教えていただけないでしょうか。

[y_waiwai]

argvの領域はシステムで取られています。それを書き換えてはいけません
で、forの条件式にどういう意味があるのかを考えてみれば。

[cgen]

そうなのですね！！以後気を付けます！！
配列argvのargc番目がNULLでなければ続けよう、と思ったのですが、、。execlの最後の引数はNULLでなければならずそこで終わりが定義されているから間違っている、ということなのでしょうか。

[y_waiwai]

argv[argc]=NULL;
としているんですから、これはNULLしかありえない。
そのforループが回ることはありえない、ってことはわかるでしょうか

Answer 2

execl等の関数は行ったきりです。コマンドの実行が終わっても戻ってきません。

連続して実行したいのなら、forkして子プロセスでexeclを実行して親プロセスでは子プロセスの終了を待ちます。
その辺を簡便に済ますなら、system関数を使うとコマンド実行後に戻ってきますので、引き続き実行できます。systemは内部でforkしてシェルをexecveしています。

argc、argvの使い方については、argv[argc]=NULL;は、配列の添え字範囲外アクセスになります。
argvの添え字の範囲は0からargc-1までです。