De forma geral tudo o que se refere aos dados deve ser feito no modelo, ainda que indiretamente. Quaisquer operações que manipulem os dados deveriam estar no modelo.
Então o método de inserir aquele dado deve ser colocado no modelo mesmo. E pode colocar o detalhe fora dele, como foi feito neste exemplo.
Imagino que a dúvida é se o controlador deve fazer toda implementação da inserção e de forma geral não deve.
Cada um com sua responsabilidade. O controlador deve dizer o que fazer e não como fazer.
Eu não tinha prestado atenção inicialmente, mas a linha Cliente c = new Cliente();
não faz o que imagina. Está criando outro objeto dentro do objeto que está manipulando. Isto não deve ser feito, a não ser que tenha um motivo para ter um cliente novo dentro do cliente, o que é altamente improvável.
Não é este cliente que deve usar para gravar o modelo, é o this
.
Acho esquisito criar um cliente novo em estado inválido.
E não me parece que esse DAO seja bem construído. Na verdade questiono o uso de DAO específico assim, mas isto é outro assunto.