forgot to close fp