avvessalom / itmo-low-level-programming Goto Github PK

ты его тут не учитываешь
for (int row = 0; row < header.biHeight; row++){
for (int col = 0; col < header.biWidth; col++){
in_bmp->data[row * header.biWidth + col] = *(struct pixel *) ((data) + sizeof(struct pixel) * (row * header.biWidth + col) + padding * row);
}
}
паддинг это просто мусор, который добавляют в конец строки чтоб она делилась на 4, то есть тебе надо скипать этот мусор

for (int row = 0; row < header.biHeight; row++){
for (int col = 0; col < header.biWidth; col++){
in_bmp->data[row * header.biWidth + col] = *(struct pixel *) ((data) + sizeof(struct pixel) * (row * header.biWidth + col) + padding * row);
}
//вот тута
}

Зачем ты выделяешь память для маленьких структур типа struct img?
in_bmp = (struct img *) malloc(sizeof(struct img));

header->bfType = 19778; // 4D42 (little-endian)

header->bfType = 0x4D42;

Я бы разделил эту функцию на части чтения хедеров и самой картинки

А в случае записи надо его учитывать
fwrite(data, 1, (out_bmp->width + padding) * out_bmp->height * sizeof(struct pixel), file);
Тут у тебя все пиксели пихаются без учета паддинга и в итоге нужные пиксели считаются мусорными

Для большей расширяемости стоит сделать функцию генерирующую struct image, чтоб если ты менял ее, было видно где надо код переписать