Возникли проблемы с программой hexdump в C - PullRequest
Новая
       6

Возникли проблемы с программой hexdump в C

0 голосов
/ 21 октября 2018

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

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

static error_t parse_opt(int key, char *arg, struct argp_state *state);
static void dump_hex(char *f, size_t l);

static struct argp_option options[] = {
    {"filepath",    'f', "PATH",    0, "uses filepath provided by user" },
    { 0 }
};

struct arguments {
    char *path;
    /* int *column_size; */
};

static error_t parse_opt(int key, char *arg, struct argp_state *state) {

    struct arguments *arguments = state->input;

    switch(key) {
        case 'f':
            arguments->path = arg;
            break;

        default:
            return ARGP_ERR_UNKNOWN;
        }

    return 0;
}

static struct argp argp = { options, parse_opt, NULL, NULL };

static void dump_hex(char *f, size_t l) {

    FILE *fd;
    size_t i;
    unsigned char *b = (unsigned char *)malloc(16 * sizeof(unsigned char));

    memset(b, 0, 16 * sizeof(unsigned char));

    if((fd = fopen(f, "r"))) {
        for(i = 0; i <= l; i++) {
            if((i % 8) == 0) {
                if(i != 0) {
                    printf("| %s\n", b);
                }
                /* print the offset */
                printf("%05lx: ", i);
            }
            /* check if ASCII is printable */
            b[i % 16] = isprint(b[i]) ? b[i] : '.';

            /* print ASCII */
            printf("%02x ", b[i]);
        }

        /* print remaining ASCII */
        printf("| %s\n", b);
    }

    fclose(fd);
    free(b);
}

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

    struct arguments arguments;
    struct stat sb;
    off_t size;

    arguments.path = "-";

    argp_parse(&argp, argc, argv, 0, 0, &arguments);

    stat(arguments.path, &sb);
    size = sb.st_size;

    dump_hex(arguments.path, size);

    return 0;
}

Вот вывод, который я получаю после передачи аргумента двоичному файлу: 

./hexdump --filepath=/tmp/a.out

212d0: 00 00 00 00 00 00 00 00 | ................
212d8: 00 00 00 00 00 00 00 00 | ................
212e0: 00 00 00 00 00 00 00 00 | ................
212e8: 00 00 00 00 00 00 00 00 | ................
212f0: 00 00 00 00 00 00 00 00 | ................
212f8: 00 00 00 00 00 00 00 00 | ................
Segmentation fault

Кроме того, любые советы о том, как сделать код более лаконичным, были бы очень полезны..

1 Ответ

0 голосов
/ 21 октября 2018

Причина, по которой вы получаете нули, заключается в том, что вы не читаете содержимое вашего файла, и b инициализируется с 0x00.Вы просто открываете и закрываете файл.

Другая проблема (это может быть причиной ошибки сегментации) состоит в том, что b имеет размер 16 байт.В некоторых местах вы используете i % 16, но не во всех (в некоторых местах вы используете i напрямую).Вы обязательно должны это проверить.

...