[C++] Obraz bmp

Question

Cześć Wam,

Chciałbym zrobić program, który wczytuje obraz bmp do programu, a następnie zmniejsza lub zwiększa konkretne wartości kolorów RGB. Pliki RGB składają się z nagłowka, w którym znajduje się rozmiar pliku, liczba wierszy, liczba kolumn).

Link do kodu źródłowego

Dla małych plików ok. 15KB wszystko działa jak należy, ale gdy wczytuję plik o rozmiarze 1MB wyświetla mi się błąd, że "Program przestał działać", nie mam kompletnie pomysłu co tu może być nie tak.

Comments

Protip (na przyszłość): dodaj symbole do debugowania do programu i uruchom program z debuggerem.

Dla GCC i GDB wygląda to np tak:

$ g++ -g -o myprogram main.cpp My program.cpp

$ gdb ./my program

(gdb) run

---

Nie trzeba nawet z debuggerem, AddressSanitizer daje wszystkie informacje na tacy. Inna sprawa, że na MinGW pewnie tak po prostu się nie da :/

 g++ -g -fsanitize=address main.cpp && ./a.out
sh: 1: cls: not found
Podaj nazwe pliku do odczytu: /home/adrian/Downloads/tux.png
=================================================================
==14689==ERROR: AddressSanitizer: stack-buffer-overflow on address 0x7ffdd52b00e8 at pc 0x55a3cdf23c8e bp 0x7ffdd52af8f0 sp 0x7ffdd52af8e0
READ of size 1 at 0x7ffdd52b00e8 thread T0
    #0 0x55a3cdf23c8d in nazwa_in(char*) /home/adrian/test/main.cpp:26
    #1 0x55a3cdf25a95 in podstawowe_dane() /home/adrian/test/main.cpp:150
    #2 0x55a3cdf260ca in main /home/adrian/test/main.cpp:187
    #3 0x7fec9a24a1c0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x211c0)
    #4 0x55a3cdf23a19 in _start (/home/adrian/test/a.out+0x1a19)

 

Answer 1

char obrazek[bajty_obrazka];

Alokujesz pamięć na stosie która jest znacznie większa od rozmiaru stosu. Coś takiego powinno się robić dynamiczną alokacją.

Poza tym, coś takiego technicznie jest nielegalne w C++ie.

main.cpp:85:31: warning: ISO C++ forbids variable length array ‘obrazek’ [-Wvla]
     char obrazek[bajty_obrazka];
                               ^

Taki Visual Studio by tego w ogóle nie skompilował.

Z innych ciekawostek, wychodzisz poza tablicę.

char bmp_we[200], bmp_out[200];
(...)
// w nazwa_in()
return bmp_in[200];

Czemu nie używasz nigdzie std::string i std::vector?

Comments

Stwierdziłem, że nazwę pliku lepiej przechowywać w tablicy char, a ta funkcja, to nie wiedziałem, jak zwrócić nazwę. Chociaż z drugiej strony w sumie rzeczywiście lepiej by było bazować na stringu. Dzięki za podpowiedź z tą alokacją - już sobie poradziłem z problemem i czytam plik co 3 bajty (stworzyłem 3 zmienne int, które czytają kolejne bajty obrazka, wrzuciłem je w pętle, która co każdy obieg zastępuje je kolejnymi wartościami odczytanymi z obrazka).