Имхо из этой процедуры открытие\закрытие файла нужно вынести.
Ведь если я захочу 10000 паролей, то прога 10000 раз откроет\закроет файл, что само по себе нехорошо.