Блог

Главная - Вопросы по программированию - CS50 восстанавливает ошибку сегментации в октябре 2020 года

CS50 восстанавливает ошибку сегментации в октябре 2020 года

#c #segmentation-fault #cs50

#c #ошибка сегментации #cs50

Вопрос:

Я новичок в программировании и начал с курса CS50 в Гарварде. Я написал некоторый код для восстановления для cs50 и попытался запустить его, но возникла ошибка сегментации.

Нужна помощь в определении того, что не так.

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

typedef uint8_t BYTE;

int main(int argc, char *argv[])
{
    if (argc != 2)
    {
        printf("Usage: ./recover imagen");
        return 1;
    }

    FILE *file = fopen(argv[1], "r");

    if (file == NULL)
    {
        printf("Usage: ./recover imagen");
    }

    char filename[8];
    FILE *img = NULL;
    BYTE bytes[512];
    int counter = 0;

    while (fread(bytes, 512, 1, file) == 1)
    {
        if(bytes[0] == 0xff amp;amp; bytes[1] == 0xd8 amp;amp; bytes[2] == 0xff amp;amp; (bytes[3] amp; 0xf0) == 0xe0 )
        {
            if (counter > 0)
            {
                fclose(img);
            }
            sprintf(filename, "i.jpg", counter);
            img = fopen(filename, "w");
            fwrite(bytes, 512, 1, img);
            counter  ;
        }
        else
        {
            fwrite(bytes, 512, 1, img);
        }
    }
    if (img == NULL)
    fclose(img);

    if (file == NULL)
    fclose(file);

    return 0;
}

Комментарии:

1. if (file == NULL) : вам также необходимо прервать программу. img = fopen(filename, "w"); вы не проверяете, имеет ли img значение NULL. В противном случае я не вижу никакой очевидной ошибки. Вы запускали это с помощью своего отладчика?

2. OT: … и второе сообщение об ошибке "printf("Usage: ./recover imagen");" неверно

3. if (img == NULL) fclose(img); Условие не имеет смысла. Вам необходимо закрыть, если указатели отсутствуют NULL .

Ответ №1:

Сосредоточьтесь на этой части.

 else
{
    fwrite(bytes, 512, 1, img);
}

Допустим, я читаю каждый 512 байт на карте памяти, не смог найти ни одного знака jpeg. И код напрямую переходит в else оператор. Давайте посмотрим, что там происходит.

 fwrite(bytes, 512, 1, img);

img здесь равен НУЛЮ (т.Е. Такой файл img не создан), и fwrite намеревается выполнить запись в несуществующий файл. Бам! Это ошибка сегментации. Если вы добавите это условие, все должно быть в порядке.

     else if (img != NULL)
    {
        // Write the data to a new jpeg file.
        fwrite(bytes, 512, 1, img);
    }