C++ not accessing method in ifstatement

113 views Asked by At

The if statement block now executes when I add the \n here:

printf("Executing the command: %s\n", cmd);

Before this addition, it didn't hit the getListofProcesses() method. The original code is shown below:

#include <stdio.h>
#include <unistd.h>
#include <cstring>
#include <stdlib.h>
#include <iostream>
#include <sys/wait.h>

char* getlistOfProcesses(const char* cmd) 
{
    FILE* pipe = popen(cmd, "r");
    if (!pipe) return (char*)"ERROR";
    char buffer[128];
    char *result = new char[1024];
    while(!feof(pipe)) {
        if(fgets(buffer, 128, pipe) != NULL)
            strcat(result, buffer);
    }
    pclose(pipe);
    return result;
}

int main(int argc, char **argv)
{   
    int P2C[2];
    int C2P[2];
    pipe(P2C);
    pipe(C2P);
    pid_t cPid = fork();
    char cmd[50];
    char* listOfProcesses = new char[1024];

    if (cPid == 0)
    {
        close(P2C[1]); 
        close(C2P[0]); 
        read(P2C[0], cmd, 50);
        if(strcmp(cmd,"LISTALL") == 0)
        {
            printf("Executing the command: %s", cmd);
            write(C2P[1], getlistOfProcesses("ps -ax -o pid,cmd"), 1024);
            close(P2C[0]);
            close(C2P[1]);
        }
    }
    else if (cPid > 0)
    {
        close(C2P[1]); 
        close(P2C[0]); 
        write(P2C[1], "LISTALL", 50);
        wait(NULL);
        read(C2P[0], listOfProcesses,1024);
        printf("%s\n",listOfProcesses); 
        close(C2P[0]);
        close(P2C[1]);
    }
    ...
    return 0;
}

I'm not sure if this is due to the way my pipes are done or if it is being accessed and I don't notice, I tried to check with a printf near the top of the method.

1

There are 1 answers

0
ross On BEST ANSWER

You need to include a newline in your format string, or call fflush(stdout)after your printf statement. See Why does printf not flush after the call unless a newline is in the format string? for more information.