exec関数で呼び出したmoreコマンドが終わらない

Question

前提・実現したいこと

C言語でunixコマンドの「ps -A | more」と全く同じ動作をさせるプログラムを作成したいです．

発生している問題・エラーメッセージ

ps -Aの内容をmoreに渡して出力することはできていますが，moreコマンドが終わりません．
（最後までEnterキーを押しても終わらない）

######変更後
moreコマンドが終了するようになりましたが，

#ps -A | more
11223 ?        00:00:00 kworker/4:3
11224 ?        00:00:00 kworker/2:0
11265 pts/9    00:00:00 ps
11266 pts/9    00:00:00 more

#./a.out
11392 ?        00:00:00 kworker/2:2
11422 ?        00:00:00 kworker/3:0
11454 pts/9    00:00:00 a.out
11455 pts/9    00:00:00 ps

となり，ps -A | more と全く同じ動きにはなりませんでした．

該当のソースコード

#include <sys/types.h>
#include <stdio.h>
#include <stdlib.h>
#include <sys/stat.h>
#include <sys/wait.h>
#include <fcntl.h>
#include <unistd.h>

#define READ 0
#define WRITE 1

#define STD_in 0
#define STD_out 1

int main(int argc, char *argv[])
{
  int fd[2];
  pid_t pid1, pid2;
  int status;

  if(pipe(fd) < 0){
    perror("pipe");
    exit(EXIT_FAILURE);
  }
  
  pid1 = fork();

  if (pid1 == 0)
  {
    dup2(fd[WRITE],STD_out);
    execlp("ps", "ps", "-A", NULL);
    close(fd[WRITE]);
  }
  else if (pid1 > 0)
  { //親
    wait(&status);
  }
  else
  {
    perror("fork");
    exit(EXIT_FAILURE);
  }
  
  pid2 = fork();
  
  if (pid2 == 0)
  { 
    dup2(fd[READ],STD_in);
    execlp("more", "more", NULL, NULL);
  }
  else if (pid1 > 0)
  { //親
    wait(&status);
  }
  else
  {
    perror("fork");
    exit(EXIT_FAILURE);
  }
  
}

変更後

#include <sys/types.h>
#include <stdio.h>
#include <stdlib.h>
#include <sys/stat.h>
#include <sys/wait.h>
#include <fcntl.h>
#include <unistd.h>

#define READ 0
#define WRITE 1

#define STD_in 0
#define STD_out 1

int main(int argc, char *argv[])
{
  int fd[2];
  pid_t pid1, pid2;
  int status;

  if(pipe(fd) < 0){
    perror("pipe");
    exit(EXIT_FAILURE);
  }
  
  pid1 = fork();

  if (pid1 == 0)
  {
    dup2(fd[WRITE],STD_out);
    execlp("ps", "ps", "-A", NULL);
  }
  else if (pid1 > 0)
  { //親
    wait(&status);
    close(fd[WRITE]);
  }
  else
  {
    perror("fork");
    exit(EXIT_FAILURE);
  }
  
  pid2 = fork();
  
  if (pid2 == 0)
  { 
    dup2(fd[READ],STD_in);
    execlp("more", "more", NULL, NULL);
  }
  else if (pid1 > 0)
  { //親
    wait(&status);
  }
  else
  {
    perror("fork");
    exit(EXIT_FAILURE);
  }
  
}

試したこと

exit関数や，return 0;を入れてみましたが，execlp("more", "more", NULL, NULL);自体が終了しないため，解決には至りませんでした．

Answer 1

原因は親プロセス側でパイプをクローズしていないためです。
コード上、execlp直後にcloseがありますが、これは子プロセス側の分岐です。親プロセス側の分岐に書くべきところを間違えたのではないでしょうか。

なぜクローズしないとまずいかはパイプの仕様に関係します。
pipe(7)manページより:

プロセスが空のパイプから読み出しを行おうとした場合、 read(2) はデータが読み出し可能になるまで停止する。

(略)パイプの書き込み側を参照しているファイルディスクリプターが すべてクローズされた後で、そのパイプから read(2) を行おうとした場合、 end-of-file (ファイル末尾) が見える (read(2) は 0 を返す)。

ここの記述にある通り、「書き込み側のパイプの口 (ファイルディスクリプタ)」が全て閉じていて初めて、読み込み側は「データ終了(eof)」を判断することができます。しかし今回は親プロセスがクローズしていないため、いつまで経っても終了と判断できません。なので、読み込みしているmoreも終われません。

蛇足ながらあともう一つ、潜在バグがあります。
最初の fork後の親のwaitがタイミング早すぎます。次のforkを呼ぶときにpsが終了できる状態にある保証はないので、お見合い状態になって先に進めなくなる恐れがあります。
このwaitは、2個目のwaitと同じタイミングまでずらす所です。

Answer 2

ps -A | more と全く同じ動きにはなりませんでした．

は何のことでしょうか？
moreが表示されないことであれば、psの完了をwaitしてからmoreを実行しているので、
ps時点ではmore未実行です。