O problema é que as suas variáveis são declaradas como char
, mas você faz a leitura como int
. Assim, a função scanf
faz a leitura de mais do que 1 byte e invade a memória da variável x2
(que como foi declarada depois, está logo em seguida na memória).
Usando o Visual Studio, por exemplo, dá pra perceber que os endereços de memória de cada uma das variáveis é seguido (x1
em 0x00ff8134 e x2
em 0x00ff8135, no meu exemplo, e com 1 byte de diferença já que as variáveis são alocadas como char
):
Dai, se você faz a leitura para x2
antes, dá pra perceber que a função altera 4 bytes (eu compilei para 32 bits).
Antes do scanf
em x2
:
Depois do scanf
em x2
(com entrada do valor 7
):
O mesmo vai valer para a leitura para x1
que você fizer em seguida, só que dai, como a memória é atualizada em 4 bytes, vai ser alterado o valor original de x2
que vem logo a seguir (observação: o endereço de início nas telas a seguir é o da variável x1
, mas ele não é mais o mesmo de anteriormente porque eu capturei as telas em uma nova execução).
Antes da leitura de um valor para x1
(com entrada do valor 2
, no meu exemplo):
Depois da leitura de um valor para x1
(perceba como a área de memória de x2
foi alterada):
É por esse motivo que a função scanf
é considerada insegura e seu uso é desaconselhado. Para executar o meu teste, por exemplo, eu tive que usar:
#define _CRT_SECURE_NO_WARNINGS
Sem esse define, o próprio compilador (no meu caso, do Visual Studio 2012) aconselha usar uma alternativa da Microsoft chamada scanf_s
.