Dynamiczne przydzielanie pamięci malloc

Question

Witam

ponieważ dowiedziałem się dzisiaj, że używanie w mallocu zmiennej, która jest wczytywana od użytkownika jest niezgodne z standardem i jedyny słuszny sposób to deklarowanie stałej wielkości w mallocu, np. malloc(5 * sizeof(int)).

A więc czy podany kod jest poprawny, bo widziałem na internecie wiele kodów używających malloca w podobny sposób.

Jeśli ten kod jest zły to istnieje jakiś inny sposób na alokacje tablicy o rozmiarze określonym przez użytkownika?

Pozdrawiam 

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

int main()
{
    int rozmiar;
    printf("Podaj rozmiar tablicy: ");
    scanf("%d", &rozmiar);

    int * tab = malloc(rozmiar * sizeof(int));

    free(tab);
    return 0;
}

Answer 1

Nie jest to prawda. Każde dane od użytkownika które wprowadzasz traktujesz jako skażone (ang. taint). Sprawdzasz zakresy akceptowane i jeśli spełniają kryteria, możesz ich używać. W Twoim kodzie jest tylko wprowadzenie rozmiaru do typu int. IMHO minimalne sprawdzenie to czy:

1. Wprowadzono typ int (a nie np. napis).
2. Nie przekraczają wartości maksymalnej.
3. Nie są ujemne (sam decyduj co zrobisz z zerem)

PS. Ogólna uwaga, sprawdź jakie problemy z bezpieczeństwem aplikacji niesie ze sobą scanf().

Comments

Dzięki za odpowiedz, nie sprawdzałem w programie poprawności danych, ponieważ chciałem przedstawić tylko główny zarys mojego pytania, czy można alokować tablice w zależności od ilości, którą poda użytkownik, ale dzięki za dokładną odpowiedź na pewno będę korzystał