Мой код fork-fifo c не работает должным образом

Question

У меня проблемы со следующим кодом. Я делаю лабораторное задание по использованию вилок (создание процессов). Это простая программа, которая должна считывать ввод с клавиатуры, а затем читать / записывать его в fifo и отображать его содержимое и записанные байты.

Когда я запускаю его, все выглядит нормально, пока я не введу какой-нибудь текст. Сообщение родительской печати отображается нормально, но сообщение дочерней печати никогда не отображается, пока я не введу свое второе сообщение, в котором всегда говорится, что записано 80 байт, хотя я знаю, что это не так, и есть куча странных специальные символы везде.

Вот исполняемый файл о том, как должна быть программа: Linux: http://www.mediafire.com/?6806v24q6lz7dpc QNX: http://www.mediafire.com/?a9dhiwmrlx2ktkp

И мой код до сих пор:

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

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

    char FifoName[] = "fifoDan";
    int fd;
    pid_t retval;
    int size_read;
    char buff[80];
    int  size_written;

    mknod(FifoName, S_IFIFO | 0666, 0);

     // Check if its not equal to zero (ie: child process = 0)
     if (retval = fork ()) {

             printf ("Parent: Waiting for writers \n");

             if(fd = open(FifoName, O_RDONLY) == -1) {
                            perror( "Could not read the FIFO" );
                    return EXIT_FAILURE;
             }

             printf ("Parent: Received a writer \n");

             do {

                             int strsize;

                             size_read = read(fd, buff, sizeof(buff));

                             printf("Parent: read %d bytes: %s \n", size_read, buff);
                             fflush(stdout);

                            strsize = strlen(buff);

                             //   put a '\0' at the end of the data
                            buff[strsize] = '\0';


                     } while(size_read > 0);


                    close(fd);
                    waitpid(retval, NULL, NULL);

                    if(unlink(FifoName) != -1) {
                            return EXIT_SUCCESS;
                    } else {
                            return EXIT_FAILURE;
                    }


    } else {


      printf ("Child pid %d waiting for readers \n", getpid ());
      fflush(stdout);

       if(fd = open(FifoName, O_WRONLY) == -1) {
                            perror( "Could not read the FIFO" );
                    return EXIT_FAILURE;
            }

      printf ("Child: Got a reader, enter some stuff:\n");
      fflush(stdout);

      while(fgets(buff, 80, stdin) != NULL) {

                     int strsize;

                     strsize = strlen(buff);

                     if(strsize < 80) {
                             buff[strsize] = '\0';
                     }

                    size_written = write(fd, buff, sizeof(buff));

                    printf ("Child: wrote %d bytes \n", size_written);
                    fflush(stdout);

             }

             close(fd);

    }


}

Answer 1

Эта строка (например) не будет делать то, что вы думаете:

if(fd = open(FifoName, O_WRONLY) == -1)

Оператор равенства имеет более высокий приоритет, чем оператор присваивания, поэтому на самом деле вы делаете следующее:

if(fd = (open(FifoName, O_WRONLY) == -1))

когда вы хотите это:

if((fd = open(FifoName, O_WRONLY)) == -1)

---

Проблема с чтением / записью в буфер заключается в том, что вы пишете весь буфер:

size_written = write(fd, buff, sizeof(buff));

Изменить, чтобы написать только фактическую строку, используя strsize вместо.