Dotaz: Únik paměti v C

[NIC5]

Příloha:

• 93838-ml-png.png (82691 bytů)

Dobrý den, měl bych dotaz ohledně úniku paměti v programu. Nevíte někdo, jak vyřešit paměť, aby se tohle nestalo. Děkuji za rady.

int lencln(char const * sin){ int len = 0;

for (int i = 0; sin[i] != '\0'; i++) { if(sin[i] >= 'A' && sin[i] <= 'Z' || sin[i] >= 'a' && sin[i] <= 'z'){ len++; } }

return len; }

char * clean(char const * sin){ int j = 0; char * str_cln = NULL;

str_cln = (char*)malloc(sizeof(char) * (lencln(sin) + 1));

if(!str_cln){ fprintf(stderr, "Out of memory!\n"); exit(-10); }

for (int i = 0; sin[i] != '\0'; i++) { if(sin[i] >= 'A' && sin[i] <= 'Z' || sin[i] >= 'a' && sin[i] <= 'z'){ str_cln[j] = sin[i]; j++; } }

return str_cln; }

int main(){ char sin[101];

scanf(" %100[^\n]s", sin);

char * strout = clean(sin);

printf("%s\n", strout);

free(strout); strout = NULL;

return 0; }

---

---

Odpovědi

[candat]

Ahoj, přepiš to do rustu, c už dnes nikdo nepoužívá právě z tohoto důvodu, nejde to zabezpečit.

[MakeIranBombedAgain❗]

Viď riadok č. 28.

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

int lencln(char const * sin)
{
	int len = 0;

	for (int i = 0; sin[i] != '\0'; i++)
		if((sin[i] >= 'A' && sin[i] <= 'Z') || (sin[i] >= 'a' && sin[i] <= 'z'))
			len++;

	return len;
}

char * clean(char const * sin){
	
	int j = 0;
	char *str_cln = NULL;

	str_cln = (char*) malloc(sizeof(char) * (lencln(sin) + 1));
	if(str_cln == NULL)
		return NULL;

	for (int i = 0; sin[i] != '\0'; i++)
		if((sin[i] >= 'A' && sin[i] <= 'Z') || (sin[i] >= 'a' && sin[i] <= 'z'))
			str_cln[j++] = sin[i];

	str_cln[j]='\0'; // !!!

	return str_cln;
}

int main(){
	char sin[101];
	
	scanf(" %100[^\n]s", sin);

	char * strout = clean(sin);

	puts(strout);

	free(strout);
	strout = NULL;
	
	return 0;
}

A ten Tvoj obrázok je výstup z akého programu?

[MakeIranBombedAgain❗]

A ten Tvoj obrázok je výstup z akého programu?

z toho samýho jeho zkompilovanýho :O ;D hele jendův tutorial na asan ;D

[MakeIranBombedAgain❗]

.......obrázok..

btw text z konzole si mužeš normálně zkopírovat když ho uplně vobyčejně voznačíš myšičkou a pak máčkneš ctrl+shift+c a nemusíš čarovat s vobrázkama :O ;D ......a mužeš taky líp poskovávat jakejže soubor kde kompiluješ by tě učitelskej nemoch doxnout :O :D ;D ;D

podobně ctrl+shift+v na vložení textu do terminálu :O ;D

[MakeIranBombedAgain❗]

Dej kód do <pre>, tohle je na prd.

Nějak jsem to otipoval a hned při kompilaci vyletí warning:

a.c: In function ‘lencln’:
a.c:8:22: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
    8 |     if(sin[i] >= 'A' && sin[i] <= 'Z' || sin[i] >= 'a' && sin[i] <= 'z'){
      |        ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
a.c: In function ‘clean’:
a.c:23:22: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
   23 |     if(sin[i] >= 'A' && sin[i] <= 'Z' || sin[i] >= 'a' && sin[i] <= 'z') {
      |        ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~

tak bych začal tím. Dále bývá zvykem přiložit vstup, na kterém to spadne (např. jestli když tady testuješ písmenka, tak musí mít vstup čísla nebo tak něco), ale naštěstí mi to spadlo na prvním co jsem zadal.

A ta chyba je podle mě v tom, že alokuješ str_cln mallocem (neinicializované), nikde do toho nenapíšeš terminující nulu a pak se to pokusíš printfnout. No a printf čte a čte a čte, až narazí na konec. Takže tam přidej na konec clean str_cln[j] = 0; nebo alokuj callocem, ten ti to vynuluje.

Plus dávám za použití satanizéru, který zjevně alokovanou paměť otravuje, protože když to zkompiluju bez něj, tak mi většinou malloc vrátí nuly (byť podle specifikace tam může být cokoli), protože heap je na začátku vynulovaný aby tam nebyla tajná data předchozích procesů, a chyba se tak hned neprojeví.

[MakeIranBombedAgain❗]

Tady je komiks kde mají přesně ten tvůj problém.

[NIC5]

Vyřešeno. Bylo to v tom '\0', jak jste radili.

[NIC5]

Vyřešeno. Bylo to v tom "\0", jak jste radili.