>>334
fin >> c;
が２回あらわれる点に問題がありますね。
今回のループは簡単なものだから、間違いはないでしょうけれども、
fin>>c にあたる部分もっと複雑な内容であれば、たとえば、あとから見たときに取りこぼしそうです。

あと、>>344 は最後の 1 バイトを取りこぼしてしまいます。
c に意味のある値が入っていても、eof() は true になります。今回の例では、
fin>>c; → v.push_back(c); → eof() 判定
の順でなければなりません。