Sim, parece redundante, está mudando um dado que não é usado depois em um objeto que logo será destruído.

Aproveitando poderia ter declarado `dado` direto no seu uso e teria um código menos redundante. E poderia ter usado um tipo no `typedef` para evitar a redundância de usar `struct`.

    typedef struct no {
        int dado;
        struct no *proximo;
    } No;

    typedef struct {
        No *inicio;
        No *fim;
    } Fila;

[Coloquei no **GitHub** para referência futura][1].

No código também não precisa mais usar `struct`.


  [1]: https://github.com/maniero/SOpt/blob/master/C/Struct/Simplification.c