[jedrek018]

Pomoże ktoś o tej porze? :)

Dlaczego free(przejsciowy); sypie "HEAP CORRUPTION DETECTED"?

http://pastebin.com/c26hfNAE

Siedzę już nad tym trochę i nie umiem nic wymyślić. :(

#programowanie

[ntskj]

@jedrek018: nie korumpuj stosu, cwaniaku :<

piszesz spod linuxa?

[jedrek018]

@ntskj: no właśnie na Windowsie siedzę. Mogę spróbować skompilować pod Linuksem.

[ntskj]

@jedrek018: valgrind jest rewelacyjnym narzędziem jeżeli chodzi o wykrywanie takich błędów, potrafi nawet wskazać free w której w funkcji tak psuje + teraz nie ściągniesz, ale na przyszłość polecam visual studio, ale jeszcze przez chwilę popatrzę

[jedrek018]

@ntskj: w VS2013 właśnie to wszystko piszę. :) Co prawda VS jest dla C++, ale dla C daje radę. Potrzebujesz jakichś dodatkowych informacji, które VS podaje?

[KrzaQ2]

@jedrek018: Ładne UB, pierw kopiujesz string, potem patrzysz czy przypadkiem wskaźnik nie jest

NULL

:

char * wartosc = (char*)malloc(strlen(gora->str) + 1);

if (stos_pusty()) return NULL;

[ntskj]

@jedrek018: przekombinowałeś robiąc sobie typedefa na wskaźnik i potem alokując przez to pamięć - dzięki temu zawsze miałeś szalone 4 bajty wskaźnika jako pamęć dla całej struktury i zabawa zaczynała się wtedy, gdy odwołujesz się do wskaźnika dol, pisząc po stosie

[KrzaQ2]

@jedrek018: to co @ntskj napisał,

sizeof(stos)

nie jest równe

sizeof(*stos)

, którego powinieneś uƶyć.

Ponadto, nie łatwiej by tak? (i bez wycieku pamięci)

char * zrzuc()

{

if (stos_pusty()) return NULL;

stos przejsciowy = gora;

gora = gora->dol;

char * wartosc = gora->str;

free(przejsciowy);

return wartosc;

}

[ntskj]

@jedrek018: uzupełnię jeszcze @KrzaQ2: o drobny szczegół - ta poprawka sprawia że program ci się nie wysypie nawet gdy masz pusty stos, bo na chwilę obecną liczysz długość ciągu o którego istnieniu nawet nie jesteś pewny

[jedrek018]

@KrzaQ2: zamieniłem kod na Twój, ale dalej wywala to samo.

[jedrek018]

@KrzaQ2: dodatkowo zauważyłem, że teraz wartosc ma nieprawidłową wartość, VS pokazuje gwiazdkę, zamiast elementu, który funkcja powinna zdjąć ze stosu. :(

[ntskj]

@jedrek018: wzorując się na pewnym słynnym forum zadaniowym odpowiem w formie kryptozagadki

[jedrek018]

@ntskj: stwierdzam, że już za późno na programowanie, zagadkę rozwiążę jutro (a właściwie to już dziś) rano.

@KrzaQ2: @ntskj: wielkie dzięki za pomoc i poświęcony czas! :)

[KrzaQ2]

@jedrek018: ugh, miało być

char * wartosc = przejsciowy->str;