Есть ли простой способ скопировать argv в C?

Question

При написании небольших утилит UNIX на C я иногда хочу взять копию argv, предоставленную main(), чтобы я мог настроить параметры перед вызовом exec(). Ниже основывается на свободно реализации из xargs BSD (1):

void run_script(char *argv[]) { 
    char **tmp, **new_argv; 
    int argc; 

    for (argc=0; argv[argc] != 0; argc++); 
    new_argv = malloc((argc + 1) * sizeof(char *)); 
    for (tmp=new_argv; *argv != 0; tmp++) { 
     *tmp = *argv++; 
     *tmp = strdup(*tmp); 
    } 

    /* call execvp(3) using new_argv */ 

    for (i=0; i<=argc; i++) 
      free(new_argv[i]); 
    free(new_argv); 
} 

Это чувствует себя более сложным, чем это должно быть. Есть ли лучший способ написать это в C? (Возможно заполнение одного буфера, который был выделен с помощью _POSIX_ARG_MAX.)

Answer 1

Я придумал другой процесс дублирования argv. Следующий пример не короче, но я думаю, что это легче читать:

void run_script(char *argv[]) { 
    int i; 
    char **new_argv; 
    char *p, *arg_buf; 
    int argc; 

    for (argc=0; argv[argc]; argc++); 
    arg_buf = malloc(ARG_MAX); 
    new_argv = calloc(argc+1, sizeof(char *)); 
    for (i=0, p=arg_buf; i<argc; i++) { 
      p += strlcpy(p, argv[i], ARG_MAX - (p - arg_buf)); 
      p++; 
    } 

    /* call execvp(3) using new_argv */ 

    free(arg_buf); 
    free(new_argv); 
} 

ARG_MAX На Linux должен быть заменен sysconf(_SC_ARG_MAX)

Answer 2

Мой C является немного ржавый, однако ....

В предположении, что ARGV [] собирается быть доступны в течение всего срока ваш скрипт, вам не нужны строки strdup, если они не будут изменены. После того, как вы выделите память (помните, чтобы добавить в argc больше, если вы хотите добавить дополнительные параметры), вы должны просто иметь возможность memcpy. Ваш new_argv будет указывать на те же строки, что и ваш исходный argv, поэтому вам нужно будет выполнить strdup, прежде чем изменять его (или нет, если вы хотите установить его в какую-либо другую строку).

Я думаю, вам нужно внимательно посмотреть на свою петлю, for (m=0, tmp.. - для начала я не думаю, что m используется где-либо еще; во-вторых, вы тестируете *argv != 0, но не меняете * argv в цикле, поэтому он будет работать вечно или вообще не будет, и, в-третьих, в начале этого цикла все ваши new_argv являются нулевыми указателями, поэтому *tmp = strdump(*tmp); попытается дублировать строка по адресу памяти 0, не имеющая ничего общего с argv?