Question

我将命令视为ls作为输入并使用popen执行命令并将结果存储在缓冲区中。但是它不会打印命令的所有内容。请帮我。 PS当整个代码处于主要状态时，它能够工作。我试过gdb但是我无法进行调试。

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



void process_command(char * command, char * buffer)
{
    int     fd[2], nbytes;
    pid_t   childpid;
    char    readbuffer[1025];
    FILE *fp = NULL;
    pipe(fd);


    if((childpid = fork()) == -1)
    {
            perror("fork");
            exit(1);
    }

    int b = 0;
    int status = 0;

    if(childpid == 0)
    {
            /* Child process closes up input side of pipe */
            close(fd[0]);
            fp = popen(command,"r");

            /* Send "string" through the output side of pipe */
          while((b = fread(readbuffer,1,1024,fp)) > 0)  
               write(fd[1], readbuffer, b);

          status = pclose(fp);

     }

    else
    {
            /* Parent process closes up output side of pipe */
            close(fd[1]);
            waitpid(childpid,&status,0);
            /* Read in a string from the pipe */
           do
          {
            nbytes = read(fd[0], buffer, sizeof(buffer));
          }while(nbytes == -1);
          buffer[nbytes] = '\0';  
    printf("Received string: %s", buffer);
    }


}


#define MAX 1024

int main(void)
{
  char command[MAX] ;
  char buffer[MAX];
  scanf("%s",command);
  process_command(command,buffer);

    return(0);
}

Answer 1

问题在于您如何阅读子进程的输出。特别是，这句话：

        nbytes = read(fd[0], buffer, sizeof(buffer));

数组buffer从main()传递，并在函数process_command()中转换为指针。所以sizeof(buffer)将指向指针的大小，而不是数组，它与sizeof(char*)相同。假设您使用的是64位系统，指针大小为8个字节。因此，您只需要读取8个字节。

将数组大小作为附加参数传递或使用MAX：

        nbytes = read(fd[0], buffer, MAX);

附加说明：

您正在使用%s读取命令，该命令无法读取空格分隔的输入。因此，如果您想运行ls /tmp，那么它就无法运行。考虑使用fgets()。
即使使用正确的缓冲区大小，您也只能读取1024个字节。因此，如果子进程的输出更长，那么它将被截断。只要有输出就更好read()并在循环中打印

在管道中使用popen

1 个答案: