Что я делаю неправильно с read() и write()?

Question

Я пытаюсь сделать некоторые программы для модуля ядра/пользовательского уровня. Я получил предложение ранее использовать fdopen(), который работал хорошо, но я выяснил, что я должен использовать open(), read() и write() вместо этого. Я прочитал страницы руководства для них и подумал, что я правильно конвертировал fopen, fgets, fputs, и моя программа компилируется, но я не получаю желаемый результат.

У меня есть таймер, если я вхожу ./userprogram -s (int) (name), например. ./userprogram -s 5 hello, через 5 секунд он будет печатать hello на консоли, общаясь с моим модулем ядра. После того, как я переключился на эти новые функции, вместо этого он печатает /lib/ld-uClibc.so.0 (и, кажется, дожидается ~ 5 секунд, несмотря на это). Мне также нужно изменить способ работы кода уровня ядра? Я думал, что просто смогу изменить программу уровня пользователя, и модуль ядра будет продолжать работать так, как раньше. Вот что я пробовал с оригинальным кодом, который работал закомментированным:

// open file 
int pFile; 
pFile = open("/dev/mytimer", O_RDWR); 
if (pFile < 0) { 
    fprintf (stderr, "mytimer module isn't loaded\n"); 
    return 1; 
} 

// Check if timer set 
if (argc >= 4 && strcmp(argv[1], "-s") == 0) { 
    lenNum = strlen(argv[2]); 
    lenName = strlen(argv[3]); 
    char *ptr = malloc(lenNum+lenName+4); 
    strncat(ptr, argv[1], 2);//flag 
    strncat(ptr," ", 1); 
    strncat(ptr, argv[2], lenNum);//timer length 
    strncat(ptr," ", 1); 
    strncat(ptr, argv[3], lenName);//message 

    /* fputs(ptr, pFile); */ 
    write(pFile, ptr, sizeof(ptr)); 
    /* 
    while (fgets(line, 256, pFile) != NULL) { 
     printf("%s", line); 
    } */ 
    while (read(pFile, ptr, sizeof(ptr)) != 0) { 
     printf("%s", line); 
    } 

Любые предложения приветствуются.

Answer 1

[I], что я преобразовал FOPEN, fgets, fputs к ним правильно

... но вы ошиблись. Не существует однострочного эквивалента write(), равного fputs(), и нет однострочного read() на основе эквивалента fgets(). Функции ввода-вывода, основанные на потоке, выполняют большую работу, которую вы должны выполнять самостоятельно, когда используете низкоуровневые read() и write().

Некоторые различия являются:

• read() не обеспечивает завершение строки. Вы должны сделать это сами.
• write() не обращает внимания на завершение строки. Если вы хотите, чтобы он остановился на строковых терминаторах, вы должны контролировать это через количество байтов, которое вы просите передать.
• read() и write() ни один из них не может передать полное количество байтов, запрошенных в одном вызове. Если вы хотите передать определенное количество байтов, вы должны быть готовы к циклу.
• read() не останавливается автоматически на любом конкретном символе, включая новую строку
• Большинство потоков обеспечивают буферизацию, невидимую для вас. Если вы хотите буферизировать свои read() и write() s, тогда вам придется обрабатывать его самостоятельно (но эти функции хорошо подходят для него).

В вашем конкретном случае, однако, вы также совершили семантическую ошибку. Третий аргумент как write(), так и read() - это максимальное количество передаваемых байтов - часто размер буфера - но sizeof(ptr) - это размер самого указателя, а не размер пространства, на которое он указывает.

Answer 2

sizeof(ptr), вероятно, 4 или 8 (то есть sizeof(char *)), а не длина данных, которые вы хотите записать.

И способ, которым вы строите данные, чрезвычайно громоздким. Почему бы не использовать один sprintf(), или, если у вас есть, asprintf(), что даже mallocs память:

char *ptr; 
int i; 

i = asprintf (&ptr, "%s %s %s", argv[1], argv[2], argv[3]); 
write (pFile, ptr, i); 
free(ptr); 

Это не сладко и кратким?