This seems like a pretty low-level place to be calling movie::find_ffmpeg() which can do a ton of path operations.  Why do you have that check here?