Функция чтения .png-изображения и преобразовании его в OpenGL-текстуру не работает. Что не так?

Question

[Павел Матвеев]

Функция чтения .png-изображения и преобразования его в OpenGL-текстуру не работает,
возвращая null в рантайме.
Код функции:

int _png_load(const char* file, int* width, int* height) {

    std::cout << file << std::endl;

    FILE* f;
    int is_png, bit_depth, color_type, row_bytes;
    png_infop info_ptr, end_info;
    png_uint_32 t_width, t_height;
    png_byte header[8], * image_data;
    png_bytepp row_pointers;
    png_structp png_ptr;
    GLuint texture;
    int alpha;

    if ((f = fopen(file, "r"))) {
        return 0;
    }
    fread(header, 1, 8, f);
    is_png = !png_sig_cmp(header, 0, 8);
    if (!is_png) {      //Здесь выбрасывает, т.к. файл, якобы, не является .png-файлом.
        fclose(f);
        return 0;
    }
    png_ptr = png_create_read_struct(PNG_LIBPNG_VER_STRING, NULL,
        NULL, NULL);
    if (!png_ptr) {
        fclose(f);
        return 0;
    }
    info_ptr = png_create_info_struct(png_ptr);
    if (!info_ptr) {
        png_destroy_read_struct(&png_ptr, (png_infopp)NULL,
            (png_infopp)NULL);
        fclose(f);
        return 0;
    }
    end_info = png_create_info_struct(png_ptr);
    if (!end_info) {
        png_destroy_read_struct(&png_ptr, (png_infopp)NULL,
            (png_infopp)NULL);
        fclose(f);
        return 0;
    }
    if (setjmp(png_jmpbuf(png_ptr))) {
        png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
        fclose(f);
        return 0;
    }
    png_init_io(png_ptr, f);
    png_set_sig_bytes(png_ptr, 8);
    png_read_info(png_ptr, info_ptr);
    png_get_IHDR(png_ptr, info_ptr, &t_width, &t_height, &bit_depth,
        &color_type, NULL, NULL, NULL);
    *width = t_width;
    *height = t_height;
    png_read_update_info(png_ptr, info_ptr);
    row_bytes = (int)png_get_rowbytes(png_ptr, info_ptr);
   image_data = (png_bytep)malloc(row_bytes * t_height * sizeof(png_byte));
    if (!image_data) {
        png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
        fclose(f);
        return 0;
    }
    row_pointers = (png_bytepp)malloc(t_height * sizeof(png_bytep));
    if (!row_pointers) {
        png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
        free(image_data);
        fclose(f);
        return 0;
    }

    for (unsigned int i = 0; i < t_height; ++i) {
        row_pointers[t_height - 1 - i] = image_data + i * row_bytes;
    }
    png_read_image(png_ptr, row_pointers);

    switch (png_get_color_type(png_ptr, info_ptr)) {
    case PNG_COLOR_TYPE_RGBA:
        alpha = GL_RGBA;
        break;
    case PNG_COLOR_TYPE_RGB:
        alpha = GL_RGB;
        break;
    default:
        printf("Color type %d not supported!\n",
            png_get_color_type(png_ptr, info_ptr));
        png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
        return 0;
    }
    glGenTextures(1, &texture);
    glBindTexture(GL_TEXTURE_2D, texture);
    glPixelStorei(GL_UNPACK_ALIGNMENT, 1);
    glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, t_width, t_height, 0,
        alpha, GL_UNSIGNED_BYTE, (GLvoid*)image_data);
    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
    glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
    glBindTexture(GL_TEXTURE_2D, 0);

    png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
    free(image_data);
    free(row_pointers);
    fclose(f);
    return texture;
}

Ошибок при сборке нет, помимо той, что fopen() может быть небезопасной функцией.
Visual Studio 2019, MSVC, Release x64.
Подключаемые компоновщиком "либы": opengl32, glu32, glew32s, , glfw3, libpng16, libpng16_static.

Comments

[SagePtr]

А что мешает выставить брейкпоинты и проверить, в каком именно месте программы функция совершает return?

[Евгений Шатунов]

Павел Матвеев , что показывает трассировка кода?
Такую большую функцию никто читать не будет. Тебе надо сопроводить функцию данными трассировки.
Где именно в этой функции происходит выход с нулевым результатом?

[Павел Матвеев]

Евгений Шатунов, Спасибо за подсказку, написал комментарий в коде, где именно функция возвращает ноль.

[Евгений Шатунов]

Павел Матвеев, при любой проблеме в первую очередь трассируй код. Анализируй его поведение при его построчном исполнении. :)
Без этого навыка у тебя исчезающе мало шансов на работу в сфере.

[Павел Матвеев]

Евгений Шатунов, Большое спасибо за совет))) Я как-то видел, что люди так делаю, но, честно говоря, не знал, как это точно называется, так что не знал как это делается, а теперь, благодаря вам, узнал/оценил/научился))) Очень удобная "штука"

Answer 1

[Павел Матвеев]

Так, всем спасибо) Ошибку нашёл сам:
png_sig_cmp() проверяет данные, считанные в "побитовом" режиме, т.е. нужно было написать fopen(file, "rb"), а я написал fopen(file,"r")

Answer 2

[Александр Ананьев]

!png_sig_cmp(header, 0, 8);
Зачем тут знак «!» ?

Comments

[Павел Матвеев]

png_sig_cmp() возвращает ноль, если header содержит байты, соответствующие png-формату, и ненулевое значение, если не содержит. А так, да, можно было ни перед png_sig_cmp(), ни в последующем условии не писать ! спереди.