Controlla i valori dei vari parametri che invii alla funzione, alla prima chiamata, e poi alla seconda

la srand(time(0)); è meglio se la togli dalla funzione, magari all'inizio del main, o comunque in una posizione tale da venir richiamata una sola volta.