Non sembra sbagliato, ma personalmente avrei diviso la parte di lettura del contenuto del PDF e la parte di creazione del JFrame per aver qualcosa di riusabile.
Segui quanto detto da andbin, niente di più facile che sia il file ad essere corrotto.

Ciao.