Не могу найти ошибки сегментации.Бывает в любом случае перед первым тиражом

Question

Включена основная, а затем реализация шестнадцатеричного и двоичного дампа, моя главная проблема связана с основной.Код на C. Он компилируется без ошибок.Мне говорят, чтобы добавить больше деталей, потому что сообщение в основном код, но я думал, что я понял свою точку зрения.Я ищу то, что вызывает ошибку сегментации.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "xbd.h"
#include "xbd.c"

int main(int argc, char *argv[]){
    int i;
    int counter = 0;
    char *addy;
    char buffer[16];

    //If the user wishes for binary output command arg 2 will be '-b'
    if(strcmp(argv[1], "-b") == 0){
        FILE *f = fopen(argv[2], "r");
        addy = (char*) f;
        //Check for valid file
        if(f == NULL){
            printf("Error: File Empty.\n");
            return(-1);
        }
        //print starting address, faults before this print
        printf("%p: ", (void*)&f);
        while((i = fgetc(f)) != EOF){
            //While there are contents in the file, dump in binary groups of 6 chars
            if(counter == 6){
                //print human readable string here
                printf(" ");
                for(int i = 0; i < 6; i++){
                    printf("%s", buffer[i]);
                }
                printf("\n");
                printf("%p: ", (void*)&addy);
                counter = 0;
            }
            buffer[counter] = binForm(i);
            counter++;
            addy++;
        }
        fclose(f);
    }
    //If not binary, output will be in Hex
    else{
        FILE *f = fopen(argv[1], "r");
        //addy = (char*) f;
        //Check for valid file
        if(f == NULL){
            printf("Error: File Empty.\n");
            return(-1);
        }
        //Print starting address, faults before this print
        printf("%p: ", (void*)&f);
        while((i = fgetc(f)) != EOF){
            //While file has contents, dump in hex groups of 16 chars
            if(counter == 16){
                printf(" ");
                //print human readable string here
                for(int i = 0; i < 16; i++){
                    printf("%s", buffer[i]);
                }
                printf("\n");
                printf("%p: ", (void*)&addy);
                counter = 0;
            }
            if(counter%2 == 1){
                buffer[counter] = hexForm(i);
                printf(" ");
            }
            else{
                buffer[counter] = hexForm(i);
            }
            counter++;
            addy++;
        }
        fclose(f);
    }
}

char hexForm(int current_byte){
    //Print hex digits for one byte
    printf("%X", current_byte);
    //If unprintable, convert to '.'
    if(current_byte < 33)
        current_byte = 46;
    return (char)current_byte;
}
char binForm(int current_byte){
    //Print binary digits for one byte
    while (current_byte) {
    if (current_byte & 1)
        printf("1");
    else
        printf("0");

    current_byte >>= 1;
    }
    //If unprintable, convert to '.'
    if(current_byte < 33)
        current_byte = 46;
    return (char)current_byte;
}

Почему это неисправно?Я попытался прокомментировать некоторые из ранних вещей, чтобы увидеть, проходит ли это, но первые несколько строк кода приводят к ошибке сегмента.Цель программы - прочитать файл, а затем изменить текст на шестнадцатеричный или двоичный кодированный вывод в зависимости от аргумента командной строки «-b».

Answer 1

Я убрал предупреждения вашего компилятора.Кажется, код теперь работает как положено.В будущем очистите предупреждения вашего компилятора и научитесь использовать gdb или другой отладчик.

Кроме того, прежде чем использовать argv [], вы ДОЛЖНЫ проверить, чтобы в командной строке действительно было достаточно аргументов.

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

char hexForm(int current_byte);
char binForm(int current_byte);



int main(int argc, char *argv[]){
    int i;
    int counter = 0;
    char *addy;
    char buffer[16];

    if(argc < 2 )  /* the actual correct usage here is left up to OP */
    {
        printf("Invalid parameters\n")
        return(0);
    }


    //If the user wishes for binary output command arg 2 will be '-b'
    if(strcmp(argv[1], "-b") == 0){
        FILE *f = fopen(argv[2], "r");
        addy = (char*) f;
        //Check for valid file
        if(f == NULL){
            printf("Error: File Empty.\n");
            return(-1);
        }
        //print starting address, faults before this print
        printf("%p: ", (void*)&f);
        while((i = fgetc(f)) != EOF){
            //While there are contents in the file, dump in binary groups of 6 chars
            if(counter == 6){
                //print human readable string here
                printf(" ");
                for(int i = 0; i < 6; i++){
                    printf("%c", buffer[i]);
                }
                printf("\n");
                printf("%p: ", (void*)&addy);
                counter = 0;
            }
            buffer[counter] = binForm(i);
            counter++;
            addy++;
        }
        fclose(f);
    }
    //If not binary, output will be in Hex
    else{
        FILE *f = fopen(argv[1], "r");
        //addy = (char*) f;
        //Check for valid file
        if(f == NULL){
            printf("Error: File Empty.\n");
            return(-1);
        }
        //Print starting address, faults before this print
        printf("%p: ", (void*)&f);
        while((i = fgetc(f)) != EOF){
            //While file has contents, dump in hex groups of 16 chars
            if(counter == 16){
                printf(" ");
                //print human readable string here
                for(int i = 0; i < 16; i++){
                    printf("%c", buffer[i]);
                }
                printf("\n");
                printf("%p: ", (void*)&addy);
                counter = 0;
            }
            if(counter%2 == 1){
                buffer[counter] = hexForm(i);
                printf(" ");
            }
            else{
                buffer[counter] = hexForm(i);
            }
            counter++;
            addy++;
        }
        fclose(f);
    }
}

char hexForm(int current_byte){
    //Print hex digits for one byte
    printf("%X", current_byte);
    //If unprintable, convert to '.'
    if(current_byte < 33)
        current_byte = 46;
    return (char)current_byte;
}
char binForm(int current_byte){
    //Print binary digits for one byte
    while (current_byte) {
    if (current_byte & 1)
        printf("1");
    else
        printf("0");

    current_byte >>= 1;
    }
    //If unprintable, convert to '.'
    if(current_byte < 33)
        current_byte = 46;
    return (char)current_byte;
}