diff cmd, двоичные файлы различаются по x и y - PullRequest
Новая
       5

diff cmd, двоичные файлы различаются по x и y

1 голос
/ 11 сентября 2011

У меня есть программа, которая принимает два файла в качестве аргумента. Первый файл должен быть скопирован во второй. Программа разветвляется на 2 дочерних элементов, первый дочерний элемент считывает файл и передает его через канал другому, затем другой дочерний элемент записывает его в файл. Предполагается, что два файла в конце будут идентичны.

Когда я запускаю diff для сравнения двух файлов, я получаю следующую ошибку: 

virtual@ubuntu:~/Documents/OSprojects$ ./parent test.txt test2.txt
virtual@ubuntu:~/Documents/OSprojects$ cat test.txt
123456789112233445566778899
virtual@ubuntu:~/Documents/OSprojects$ cat test2.txt
123456789112233445566778899
virtual@ubuntu:~/Documents/OSprojects$ diff test.txt test2.txt
Binary files test.txt and test2.txt differ
virtual@ubuntu:~/Documents/OSprojects$

Как вы можете видеть, они оба одинаковы, но разница показывает, что они разные. Очевидно, это просто то, чего я не понимаю в diff cmd. Любая помощь будет оценена.

По какой-то причине я считаю, что созданный мной файл - это двоичный файл, тогда как первый файл - нет, но я не знаю, почему это двоичный файл. Я считаю, что это может быть связано с этой строкой кода: 

    write(1, buf, BUF_SIZE);            //write to buffer
    memset(buf, '\0', BUF_SIZE);

У одного из детей это записывает в буфер, а я очищаю буфер. Я неправильно очищаю этот буфер?

Вот результат cat -e: 

virtual@ubuntu:~/Documents/OSprojects$ cat -e test2.txt
123456789112233445566778899$
^@^@^@^@virtual@ubuntu:~/Documents/OSprojects$

Вот результат cmp: 

virtual@ubuntu:~/Documents/OSprojects$ cmp test.txt test2.txt
cmp: EOF on test.txt
virtual@ubuntu:~/Documents/OSprojects$

Я считаю, что это моя проблема, как я могу очистить этот буфер, чтобы он не выбрасывал их в конце?

ВСЕ МОЙ КОД ::

Родитель: 

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

#define BUF_SIZE 16

void exitWithError(char* errorMsg, int exitWith);       //generic error out function
void launch_writer(const char* pathname, char* const argv[], int pfd[]);    
void launch_reader(const char* pathname, char* const argv[], int pfd[]);   

int main(int argc, char* argv[]){

//making the pipe
int pfd[2];
if(pipe(pfd) == -1) //test pipe creation
    exitWithError("PIPE FAILED", 1);

//forking
pid_t reader_child_pid;
pid_t writer_child_pid;

//args for each fork
char *args_1[] = {"reader", argv[1], (char *) 0};
char *args_2[] = {"writer", argv[2], (char *) 0};

if((writer_child_pid = fork()) == -1) {
    exitWithError("WRITER FORK FAILED", 1);
}
else if (writer_child_pid == 0) {       //first child comes here
    launch_writer("./writer", args_2, pfd);
}
else if ((reader_child_pid = fork()) == -1) {
    exitWithError("READER FORK FAILED", 1);
}
else if (reader_child_pid == 0) {       //second child comes here
    launch_reader("./reader", args_1, pfd);
}

//parent picks up here

//close off pipe from parents end
close(pfd[0]);
close(pfd[1]);


//wait for all processes to exit before ending
for(;;) {
    if(wait(NULL) == -1){
        if(errno == ECHILD)
            exit(0);
        else {
            exitWithError("WAIT ERROR", 1);
        }
    }
}
}

void exitWithError(char* errorMsg, int exitWith) {
perror(errorMsg);
exit(exitWith);
}

void launch_writer(const char* pathname, char* const argv[], int pfd[]) {
dup2(pfd[0], 0);
close(pfd[1]);
close(pfd[0]);
execve(pathname, argv, NULL);
perror("execve failed");
}

void launch_reader(const char* pathname, char* const argv[], int pfd[]) {
dup2(pfd[1], 1);
close(pfd[1]);
close(pfd[0]);
execve(pathname, argv, NULL);
perror("execve failed");
}

Ребенок 1: 

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

#define BUF_SIZE 16

int main(int argc, char* argv[]){
//Opens file to be read from
int inFile = open(argv[1], O_RDONLY);

//declaring variables
char buf[BUF_SIZE]; //temp hold whats read/written
int read_test;      //check if EOF
for(;;) {
    read_test = read(inFile, buf, BUF_SIZE);    //read from file
    if(read_test == 0) //eof
        break;
    write(1, buf, BUF_SIZE);            //write to buffer
    memset(buf, '\0', BUF_SIZE);
}
close(inFile);
exit(0);
}

Ребенок 2: 

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

#define BUF_SIZE 16

int main(int argc, char* argv[]){
//Opens a file for reading/writing, if exists then truncates, otherwise makes new one
//with correct permissions
int wri_inFile = open(argv[1], O_WRONLY | O_CREAT | O_TRUNC , S_IRUSR | S_IWUSR);
if(wri_inFile == -1)
    perror("ERROR OPENING FILE");

//declaring variables
char buf[BUF_SIZE];  //to store what is read in/written out
int read_test;      //test if EOF
for(;;) {
    read_test = read(0, buf, BUF_SIZE); //read from buffer
    if(read_test == 0) //eof
        break;
    write(wri_inFile, buf, BUF_SIZE);   //write to file

}
close(wri_inFile);
exit(0);
}

1 Ответ

1 голос
/ 11 сентября 2011

Вы не проверяете (и не используете) длину прочитанных данных.Поэтому ваши данные дополняются мусором.

Должно быть прочитано фактических байтов данных (read_test): 

read_test = read(0, buf, BUF_SIZE); //read from buffer
if(read_test == 0) //eof
    break;
write(wri_inFile, buf, BUF_SIZE);   //write to file
-----------------------^^^^^^^^

То же самое относится и к другому дочернему элементу.Вам также следует проверить условия ошибки.

...