The check is definitely unnecessary right now. I'd expect that to change when support for more file formats is introduced, but I guess there is an argument for getting rid of the Coverity warning in the meantime, especially considering that code freeze is coming soon.
Can you also get rid of the `buffer` initialization at the top of the function while at it? It should be unnecessary as well.