Porque é que o meu array de strings está a guardar lixo?

[joanacs]

typedef struct data_cliente client;
struct data_client {
    char *subscription_list;
};

void save_subscription(char topic_chosen[50]){

    client c;
    int j;

    c.subscription_list = malloc(strlen(c.subscription_list) + 1);
    if(c.subscription_list == NULL){
        printf("\n[ERROR] Memory allocation.");
        exit(0);
    }
    else{
        strcpy(&c.subscription_list[strlen(c.subscription_list)], topic_chosen);
    }   

    for(j=0; j<strlen(c.subscription_list); j++)
        printf(" %s", &c.subscription_list[j]);
}

Eu estou a tentar guardar uma string num array dinâmico de strings mas estou a ter alguns problemas. Eu só quero alocar algum espaço para uma string de cada vez e salvá-la, para que da próxima vez possa adicionar outra string ao array, mas manter as outras já adicionadas. O meu código neste momento está a salvar lixo juntamente com a string que eu pretendo e se eu tentar adicionar uma segunda string ao array, a primeira desaparece. Alguém me pode ajudar?

[Rui Carlos]

Provavelmente o que pretendes é usar um array de strings e não uma única string.

Adicionalmente, sempre que alocas novas memória e queres manter os dados antigos, tens que copiar os dados antigos para a nova regiam de memória (e já agora, convém libertar a memória antiga).