Как мне правильно соединить три строки и сделать системный вызов?

Question

[Ирина]

Добрый день. Есть вот такой код на си:

char filenames[1000];
        memset(filenames, 0, 1000);
        printf("Enter filenames:\n");
        read(0, filenames, 1000);
        printf(filenames);
        char* part1 = "file ";
        char* part2 = "| grep 'C source' | wc -1";
        char* result = malloc(strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        memset(result, 0, strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        memcpy(result, part1, strlen(part1));
        memcpy(result, filenames, strlen(filenames));
        memcpy(result, part2, strlen(part2));
        printf(result);
        system(result);

Задача следующая: надо получить на вход у пользователя имена файлов через пробел. Дальше я хочу встроить эти имена в запрос "file | grep 'C source' | wc -1", а дальше сделать вызов этой команды с помощью system. К сожалению, у меня это не выходит. Я пыталась использовать strcat, но тоже ничего не выходит.

Answer 1

[res2001]

Когда делаете memcpy указатель на result надо смещать на размер предыдущей скопированной строки. Сейчас вы просто копируете следующую строку, затирая предыдущее копирование.

memcpy(result + strlen(part1), filenames, strlen(filenames));

strcat должен нормально отработать, он делает по сути то же самое.
При копировании строк лучше использовать strcpy (или strncpy), она копирует и завершающий 0. memcpy предназначена для копирования памяти произвольного содержимого.

Comments

[Ирина]

переписала код так:

char filenames[1000];
        memset(filenames, 0, 1000);
        printf("Enter filenames:\n");
        read(0, filenames, 1000);
        char* part1 = "file ";
        char* part2 = "| grep 'C source' | wc -1";
        char* result = malloc(strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        memset(result, 0, strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        memcpy(result, part1, strlen(part1));
        memcpy(result + strlen(part1), filenames, strlen(filenames));
        memcpy(result + strlen(part1) + strlen(filenames), part2, strlen(part2));
        printf(result);
        system(result);

Получаю такой результат:

Enter filenames:
nginx.info abc.c
file nginx.info abc.c
nginx.info: ASCII text
abc.c:      C source, ASCII text
sh: 2: Syntax error: "|" unexpected

[res2001]

Ирина, Поизучал немного вопрос.
1.Функция read возвращает введеную строку вместе с символом перевода строки (вы ведь нажимаете Enter когда вводите файл, этот Enter попадает в filenames то же). Нужно вручную устанавливать в 0 последний символ в filenames после ввода.
2.Функция system() выполняет /bin/sh -c <command>.
Что бы в этой конструкции ваш конвеер отработал правильно все команды нужно заключить в кавычки (двойные или одинарные). Строку 'C source' лучше привести к виду \"C source\" экранировав внутренние двойные кавычки
3.Даже все поправив мне не удалось заставить system выполнить конвеер. Одиночную команду он выполняет, конвеер не хочет. По уму нужно посмотреть как реализован system в glibc, но на это сейчас нет времени.
Немного погуглив нашел, что народ реализует подобную схему не используя system, а организуя конвеер с помощью fork и execlp. Пример реализации можно посмотреть тут

[Ирина]

res2001, спасибо большое)

[res2001]

Что-то я замудрил. На самом деле всю команду не нужно заключать в кавычки.

В общем мне удалось заставить работать эту конструкцию после выполнения п.1
Рабочий код:

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <unistd.h>

int main(int argc, char **argv)
{
        char filenames[1000];
        memset(filenames, 0, 1000);
        printf("Enter filenames:\n");
        read(0, filenames, 1000);
        filenames[strlen(filenames) - 1] = 0;
        const char part1[] = "file ";
        const char part2[] = " | grep 'C source' | wc -l";
        char* result = malloc(strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        memset(result, 0, strlen(part1) + strlen(filenames) + strlen(part2) + 1);
        strcpy(result, part1);
        printf("len filenames = %d\n", strlen(filenames));
        strcat(result, filenames);
        strcat(result, part2);
        printf("Command line: %s\n", result);
        fflush(stdout);
        system(result);
}