Veja a linha:
cin >> n;
A variável n
é do tipo char*, então o operador >> pensa que há um array alocado ali, e lê coisas que não cabem no buffer ("-1" tem 2 chars). Nem há buffer, porque você alocou dinamicamente apenas 1 char (para treinar, OK, num programa de verdade, parece sem propósito).
Sugiro que troque então por:
cin >> *n;
ou
scanf("%c", n);
Se digitar -1, a linha acima (qualquer uma) lerá apenas o "-", sem gravar fora do espaço alocado.
Outra coisa: &*dinheiro
é meio inútil. Os operadores & e * se cancelam (na ausência de operator overloading), e seria o mesmo que escrever apenas dinheiro
.
E uma importante lição no mundo do C e C++: o fato do compilador aceitar está muito vagamente relacionado com o fato do programa estar correto.