What is wrong with this idea

public static void removeFirstNameDuplicates(Map<String, String> map) {

        HashMap<String,String> copy = new HashMap<String, String>(map);

        for (Map.Entry<String,String> pair : copy.entrySet()) {
            String value = pair.getValue();
            removeItemFromMapByValue(map,value);
        }
}