uniqとwc、システムコールを用いてtxtファイルの連続していない行を数える

Question

“uniq < tmp.txt | wc –l”を実行するプログラムを、システムコールと uniq および wc を利用して作成せよ。システムコールとして、fork、exec 族、pipe、dup（もしくは dup2）を、その本来の目的に即して用いること。また、“execlp("bash", "bash", "-c", "uniq < tmp.txt | wc -l", NULL);”のようにシェルで当該コマンドを実行させるような構成は認められない。
tmp.txt
0
0
1
1
1
2
0
2
が存在した時に
./uniqwc tmp.txt
とすると、実行結果が5となる。

以上が与えられている問題です。

自分の書いたコードは以下のもので、execlpを使用してはいけない？ことを知らずに行ってしまいました。動作としては上手くいっています。
そこで、execlpを使わずに書き換える場合、どのような手法があるのか教えていただきたいです。
該当するページでもいいので参考させていただきたいです。
よろしくお願いします。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/uio.h>
#include <sys/wait.h>
#include <fcntl.h>
#define N 256

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

    ifd = open(argv[1], O_RDONLY);
    if(ifd == -1){
        perror("open");
        exit(1);
    }

    if(pipe(fd) == -1){
        perror("pipe");
        exit(1);
    }

    pid = fork();
    if(pid < 0){
        perror("pid");
        exit(1);
    }
    
    if(pid > 0){
        // 使わないファイルを閉じる
        close(fd[0]);
        // stdoutをパイプの入力
        dup2(fd[1], STDOUT_FILENO);
        close(fd[1]);
        // stdinをtmp.txt
        dup2(ifd, STDIN_FILENO);
        close(ifd);
        execlp("uniq", "uniq", NULL);
    } else {
        // 使わないファイルを閉じる
        close(fd[1]);
        close(ifd);
        // stdinをパイプの出力
        dup2(fd[0], STDIN_FILENO);
        close(fd[0]);
        execlp("wc", "wc", "-l", NULL);
    }

    if(close(ifd) == -1 || close(fd[0]) == -1 || close(fd[1]) == -1){
        perror("close");
        exit(1);
    }

    return 0;
}

Comments

[RiaFeed]

execlpを使うなではなく、execlpからbashとかのシェルを使う(経由する)なと取れるっぽいから大丈夫なのでは？
出題者に確認する必要はあるけど

[nagano-yuhi]

その線も考えてみたのですが、不安になり変更法を聞いてみました。
まだ提出までに時間があるので、伺ってみたいと思います。
ご指摘ありがとうございます。

[otn]

execlpを使うなとはどこにも書いてないので、このままで良いかと。
むしろ、exec 族を使って良いと書いてあるというか、exec 族使えと言うことですね。
execlpを使ったコードを書いている以上、execlpがexec 族であることを知らないと思えないのですが。

[nagano-yuhi]

回答ありがとうございます。
授業のプリントを参考にして書いたので、その辺の知識が浅かったと思います。
ご指摘ありがとうございます。

Answer 1

exec族には、execl, execlp, execle, execv, execvp, execvpe がありますが、
本当のシステムコールは execve なので、それに置き換えてみました。

-		execlp("uniq", "uniq", NULL);
+		char *argv[] = { "uniq", NULL };
+		execve("/usr/bin/uniq", argv, NULL);

-		execlp("wc", "wc", "-l", NULL);
+		char *argv[] = { "wc", "-l", NULL };
+		execve("/usr/bin/wc", argv, NULL);

Comments

[nagano-yuhi]

回答ありがとうございました。

自分でもこのあたりの事を調べたいと思います。