Se usou uma vez, viu o aviso e continuou usando, está cometendo um erro.
Buffer overflow
A função gets()
recebe uma variável que funciona como um buffer. Ou seja, uma variável é criada e passada para a função que colocará um valor nela. Quando a variável é criada ela tem um tamanho específico. Essa função não tem nenhuma medida para garantir que o valor não ultrapasse o tamanho da variável. A linguagem C por si só não possui mecanismos que garantem o limite do tamanho do objeto contido na variável.
O estouro do buffer ocorre quando o valor obtido (na leitura do dispositivo) é maior que o tamanho reservado para a variável. Então a função escreve em uma área que está reservada para outra coisa. Podendo escrever onde não deve ela permite que a execução seja adulterada e o código acabe fazendo algo inesperado, abrindo brechas de segurança.
Os "hackers" se aproveitam desta falha para enviar dados que eles sabem que vão sobrescrever memória de forma indevida. Entendendo as posições de memória que a aplicação estará usando em determinada situação ele pode mandar um código ou modificar um dado que executará um código indevido que não faz parte do funcionamento esperado na aplicação, como por exemplo habilitando um login mesmo sem uma senha correta.
Solução
A grande vantagem da função fgets()
e outras semelhantes é que um de seus parâmetros diz o tamanho do buffer e a função garante que não ultrapassará esse limite, mesmo que o valor que ela esteja recebendo seja maior. Desta forma o dado não pode interferir em outras áreas da memória.
Outras funções perigosas
Cuidado também com (segue a alternativa "segura" já existente no padrão):
strcpy()
-> strncpy()
strcat()
-> strncat()
strlen()
-> strnlen()
strcmp()
-> strncmp()
strdup()
-> strndup()
wcscpy()
-> wcsncpy()
wcslen()
-> wcsnlen()
sprintf()
-> snprintf()
Outras que podem ser ruins em algumas situações: - scanf()
, getwd()
, realpath()
atoi()
, memcpy()
, strtok()
,
Algumas dessas funções possuem variações perigosas também. Note que a alternativa não é garantidamente segura, elas apenas possuem um mecanismo um pouco melhor, elas não substituem o uso correto delas e verificações adicionais.
Algumas são perigosas e propensa a estouro de buffer apenas se mal usadas. A gets()
realmente é perigosa não importando o que o programador faça.
Obviamente não arrisco dizer que esta é uma lista completa.
Coloquei no GitHub para referência futura.