Signed-off-by: Rémi Denis-Courmont <rdenis@simphalempin.com>

I am not sure that the reverted part was really intented.
Reported by Jonathan Brossard, iViZ Techno Solutions Pvt. Ltd.

host->i_url was read without protection. We also now use the reference
count as the "death" variable, rather than vlc_object_alive() which
simplifies locking.

By the way, its earlier (<= 0.8.6) incarnation were buggy.

By the way, the Win32 video outputs could probably go through some code
factorization. Don't ask me.

(Since we are always dealing with a pipe, it cannot return EINVAL)

Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

Unbreaks start-up on WinCE.
Signed-off-by: Rémi Denis-Courmont <rdenis@simphalempin.com>

Pointed out by linkfanel.

Once the input main loop is done (or if input init fails), input
controls are not popped anyway. So the input callbacks are no-ops.
There is no point in postponing cleanup.