In quel modo si leggono i file binari non quelli di testo e comunque è inutile quel new ifstream quando puoi usare direttamente:
codice:
ifstream ifi(name.c_str());
senza lasciare memory leaks ovunque.