C Конкатенация строки во время цикла

Question

Я пытаюсь объединить часть структуры с шестнадцатеричными значениями. Я пробегаю каждый байт в цикле и конвертирую в hex, тогда я хочу объединить все шестнадцатеричные строки в одну длинную строку.

Однако, я заканчиваю только одним значением в конце цикла. По какой-то причине строка не конкатенируется правильно. Любая идея, что я делаю неправильно?

typedef struct OPTIONS_STR 
{ 
    int max; 
    int printName; 
} OPTIONS; 

void set_default_options(OPTIONS *options) 
{ 
    options->max = -1; 
    options->printName = 0; 
} 

void do_file(FILE *in, FILE *out, OPTIONS *options) 
{ 
    char ch; 
    int loop = 0; 
    char buf[81]; 
    buf[0] = '\0'; 
    int sz1; 
    int sz2; 
    int sz3; 

    int seeker = offsetof(struct myStruct, contents.datas); 

    //find total length of file 
    fseek(in, 0L, SEEK_END); 
    sz1 = ftell(in); 

    //find length from beggining to struct beginning and minus that from total length 
    fseek(in, seeker, SEEK_SET); 
    sz2 = sz1 - ftell(in); 

    //set seek location at beginning of struct offset 
    fseek(in, seeker, SEEK_SET); 

    sz3 = sz2 + 1; 
    char buffer[sz3]; 
    char msg[sz3]; 

    buffer[0] = '\0'; 

    while (loop < sz2) 
    { 
     if (loop == sz2) 
     { 
      break; 
     } 

     fread(&ch, 1, 1, in); 
     sprintf(msg, "%02X", (ch & 0x00FF)); 
     strcpy(buffer, msg); 

     ++loop; 
    } 
    printf("%s\n", buffer); 
} 

int main(int argc, const char * argv[]) { 

    OPTIONS options; 
    set_default_options(&options); 

    const char *current = "/myfile.txt"; 
    FILE *f = fopen(current, "rb"); 
    do_file(f, stdout, &options); 
    fclose(f); 

}; 

Answer 1

Использование strcat вместо strcpy. Это должно решить вашу проблему.

Для эффективности посмотреть в использовании указателя записи как char *p = buffer и продвигать позицию записи с чем-то вроде p += sprintf(p, "%02X", (ch & 0x00FF))

Кроме того, ваш if(loop == sz2) break чека является бесполезным дубликатом while(loop < sz2) проверки. Цикл while не будет выполняться, если loop равен или больше sz2.

Также интересно, почему вы используете fread, когда вам нужен только один символ. fgetc или getc, кажется, лучший выбор.

Кроме того, независимо от того, используете ли вы fread или getc, вам необходимо проверить, в конце файла. Что делать, если файл не содержит sz2 байтов? Потому что все современные системы многопроцессорны и многопользовательские, поэтому кто-то может сократить файл после вызова ftell. Вы никогда не должны принимать вещи, потому что, даже если вы просто проверили это, это может измениться. Создание этого предположения является причиной ошибок TOCTTOU (время проверки на время использования).

Answer 2

В do_file() вы копируете шестнадцатеричное значение для одного байта в цикле while. Таким образом, вы должны перейти к следующему байту массива символов buffer с каждой итерацией цикла while, то есть buffer++ или strcpy(buffer[loop], msg);