C Программирование - execlp () помочь?

Question

Я создаю оболочку в C для целей обучения, и до сих пор я дошел до того, что вы можете ввести строку с помощью fgets (), строка разбивается на «порции», а затем эти порциипередаются в execlp ().Первый блок - это имя команды, а последующие блоки - это аргументы команд.

Все работает нормально, кроме вызова execlp ().Но я не вижу, что я делаю не так, все это выглядит законным для меня, согласно страницам руководства!

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

#define MAX_CHUNKS 10

/*==========================================================================
 *  Given a string, Break it down into chunks. Separated by ' ', skipping \n
 * ========================================================================*/
int break_down_string(char *input_string, char *pointer_array[MAX_CHUNKS])
{
        char *p = input_string, buffer[100]={0};//Initialize buffer to zero's.
        short int index = 0, space_count = 0, i;


    strncat(p, " ", 1);

    while (*p != '\0')
    {   
        if (index == MAX_CHUNKS) break; //End if MAX_CHUNKS chunks taken from string.
        if (*p == '\n'){ //Skip newline characters.
            p++;
            continue;
            }

        if (*p == ' ') //Space Detected 
        {
            if (space_count == 0)
            {
                pointer_array[index] = (char *)malloc(sizeof(char) * strlen(buffer) +1);
                strncpy(pointer_array[index], buffer, strlen(buffer));
                strncat(pointer_array[index], "\0", 1);
                bzero(buffer, sizeof(buffer));
                index++;
            }
            space_count = 1;
        }
        else //Non-Space Detected
        {
            if (space_count > 0) space_count = 0;
            strncat(buffer, p, 1);
        }
        p++;
    }

pointer_array[index] = NULL; //Set end pointer to NULL for execlp().

return 0;
}



/*--------------------------------MAIN()-----------------------------------*/
int main(void)
{
    char buffer[100];
    char *pointer_array[MAX_CHUNKS]; //Array which will hold string chunks

    fgets(buffer, sizeof(buffer), stdin); 

    break_down_string(buffer, pointer_array);

    if (fork() == 0)
    {
        printf("Child process!\n");
        execlp(pointer_array[0], (pointer_array+1), NULL);
    }
    else
    {
        printf("Parent process!\n");
    }

return 0;
}

Помощь будет принята с благодарностью, я действительно застрял здесь!

Answer 1

Это не правильно:

char *pointer_array[MAX_CHUNKS];
execlp(pointer_array[0], (pointer_array+1), NULL);

execlp объявлен как int execlp(const char *file, const char *arg, ...);.Предупреждение должно прояснить, что вы не можете передать char ** там, где ожидается char *.

---

Лично я предпочитаю execvp довольно сильно.Он также позволяет передавать много аргументов новому процессу.

/* Make sure the last element of pointer_array is NULL. */
execvp(pointer_array[0], pointer_array);

Вы также можете попробовать:

execlp(pointer_array[0], pointer_array[1], NULL);