Quel codice non va bene perchè non copia la lista ma solo i riferimenti ad essa.
Nel costruttore di copia devi scorrere gli elementi eseguendo, di volta in volta, una push (e quindi allocazione di memoria per l'elemento copiato)...