O array de char
pode dar melhor performance, afinal evitará uma alocação dinâmica de memória, que é algo relativamente caro. Não importa se o tamanho será determinado em tempo de compilação ou de execução.
No exemplo da pergunta seria melhor usar essa forma, mas nem sempre isso é possível.
Segurança e confiabilidade
Segurança é sempre obtida sabendo o que está fazendo, entendendo como o computador funciona, como a linguagem trabalha, os detalhes da API que está usando, etc. O uso de array no lugar da alocação dinâmica nem ajuda nem prejudica a segurança diretamente. Não há nada em um recurso ou outro que impeça o estouro de uso de memória. Em C é um problema do programador tratar isto.
O uso de memória dinâmica costuma ser menos confiável. Não por ela em si, mas porque os programadores costumam errar mais no seu uso, e o que não é tão confiável pode ser menos seguro, mas é algo indireto.
Quando usar
Normalmente a alocação do array é feita na stack - por isso é rápido - o que não permite ter um array muito grande (muito menos se o tamanho não puder ser determinado na compilação ou pelo menos garantido que ele não será muito grande), nem que ele sobreviva ao final de uma função (ou escopo), então há casos que a alocação no heap - com malloc()
- é a única solução viável.
Mesmo nos casos da alocação ser em uma estrutura, pode fazer ela ficar grande demais, e nem sempre é o que se deseja, principalmente porque praticamente obrigaria a sua alocação no heap (alocação dinâmica), o que nem sempre é desejável.
No fundo a diferença entre o array e a alocação dinâmica, está mesmo só na alocação, uma não oferece mais recursos que outra. Depois de alocado, dá na mesma, a linguagem não diferencia uma da outra.
O padrão é sempre usar a forma mais simples, que é a alocação do array, a não ser que exista um motivo para alocar dinamicamente. Isto não é otimização prematura, pelo contrário, a alocação dinâmica deve ser evitada sempre que possível, sempre que ela não trouxer problemas específicos. Ela é mais simples, mais confiável e mais rápida, só tem vantagens quando o que precisa não será limitado pela sua característica.
Conclusão
Olhando por cima, no geral este código é mal escrito e não atende padrões modernos. Aparentemente pode ter sido escrito por quem ainda não conhece a linguagem direito ainda, até por outros erros.
Veja mais sobre a decisão (é C++, mas essencialmente é a mesma coisa).
Coloquei no GitHub para referência futura.