Este código tem alguns problemas.

- Não tem porque usar `malloc()` se a área que a *string* deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como `const char *` e não `[tamanho]`. Tem que ser bem mais cuidadoso quando faz isto. O que inclusive está vazando memória. Em grande volume isto seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
- Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o *array*. Escreva diretamente no *array*, sem criar nada intermediário, inicializar ou copiar.
- As *strings* possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da *string* do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

    #include <string.h>
    #include <stdio.h>
    
    typedef struct {
        char nome[81];
        char ano[5];
        char diretor[81];
    } Filme;
    
    void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
        strcpy(filme->nome, arg1);
        strcpy(filme->ano, arg2);
        strcpy(filme->diretor, arg3);
    }
    
    void carregar(Filme filmes[]) {
        analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
        analise(&filmes[1], "teste", "998", "bar");
        analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
    }
    
    int main() {
        Filme filmes[1000];
        carregar(filmes);
        printf("\nmain:\n");
        for (int i = 0; i < 3; i++) {
            printf("- Nome:    %s\n", filmes[i].nome);
            printf("- Ano:     %s\n", filmes[i].ano);
            printf("- Diretor: %s\n", filmes[i].diretor);
            printf("----------------\n");
        }
    }