C言語でプロセス生成してファイルへ書き込み

fork()で子プロセスを生成して"ls -al"を実行し、その実行結果を親に流して親は子プロセスから読み取ったデータをファイルへ書き込んでいるのですがwhile(read(fd[0], buf, sizeof(buf))で止まってしまいます。正しいプログラムにするためにはどのように修正すればよいでしょうか。

c
1#include <stdio.h>
2#include <string.h>
3#include <stdlib.h>
4#include <unistd.h>
5#include <sys/types.h>
6#include <sys/wait.h>
7#include <fcntl.h>
8
9int main(){
10    int fd[2];
11    pipe(fd);
12    if(fork() == 0){
13        close(1);
14        dup(fd[1]);
15        char *cmd[] = {"ls", "-a", "-l", NULL};
16        execvp(cmd[0], cmd);
17    }else{
18        int fp = open("test.txt", O_WRONLY | O_CREAT);
19        if(fp == -1) perror("open");
20        char buf[1024];
21        close(0);
22        dup(fd[0]);
23        memset(buf, '\0', sizeof(buf));
24        while(read(fd[0], buf, sizeof(buf))){
25            write(fp, buf, sizeof(buf));
26            memset(buf, 0, sizeof(buf));
27        }
28        close(fp);
29        wait(NULL);
30    }
31    return 0;
32}

episteme

2019/05/23 13:10

"止まる"とは如何なる意味か。プログラムが停止するのか? その行から次に行かなくなるのか?

行動規範の内容に同意します

回答2件

ベストアンサー

最小限の変更を加えるとこんな感じで。

C
1#include <stdio.h>
2#include <string.h>
3#include <stdlib.h>
4#include <unistd.h>
5#include <sys/types.h>
6#include <sys/wait.h>
7#include <fcntl.h>
8
9int main(){
10    int fd[2];
11    pipe(fd);
12    if(fork() == 0){
13        close(fd[0]); // 使わない側をクローズ
14        close(1);
15        dup(fd[1]);
16        char *cmd[] = {"ls", "-a", "-l", NULL};
17        execvp(cmd[0], cmd);
18    }else{
19        close(fd[1]); // 使わない側をクローズ
20        int fp = open("test.txt", O_WRONLY | O_CREAT, 0644); // モード指定
21        if(fp == -1) perror("open");
22        char buf[1024];
23        close(0); // fd[0]から読むのならこのclose&dupは不要だが
24        dup(fd[0]);
25        int n; // readしたバイト数
26        while(n=read(fd[0], buf, sizeof(buf))){
27            write(fp, buf, n);
28        }
29        close(fp);
30        wait(NULL);
31    }
32    return 0;
33}

投稿2019/05/23 13:34

otn

総合スコア84555

test.txt は誰が作った? 子プロセスはそんなファイル作ってないぞ?

ひょっとしてやりたいことは popen か?

Windowsだとこんな↓感じ。

C
1#include <stdio.h>
2#include <stdlib.h>
3
4
5int main() {
6  char line[256];
7  FILE* fp = _popen("dir /B", "r");
8  while ( fgets(line, 256, fp) ) {
9    printf("%s", line);
10  }
11  _pclose(fp);
12  return 0;
13}