"end" label is only reached if psz_eltname is NULL.

If subsequent malloc or realloc failed, free would be called on a dangling
pointer.

close #14006

This is better than leaking them to the DNS resolver. This also avoids
confusing error messages if live555 is missing or failing.

refs #14193

refs #14193

refs #14193

p_owner->b_paused can only be written by the owner, so the owner thread
does not need to lock to read it. (However, locking is necessary to
write it from the owner thread, or to read it from the decoder thread.)

This removes small hacks surrounding this corner case.

Do not rely on allocating a block for draining the decoder and output.

All audio decoders support the *pp_block == NULL case, and return NULL.
This enables audio decoders to drain cleanly.

DecoderPlayAudio() has no effects with a NULL block.

MainMenus p_intf can be already accessed before
applicationWillFinishLaunching was called.

This should fix quite some crashes reported by the users.

There is nothing special about exiting at this level. The decoder
should discard its buffers and return to reading the input FIFO at the
earliest (where it gets canceled). This is the same as flushing.

See also 0fde3bea.

See also 0fde3bea.

This fixes an assertion failure.
Regression from 62140018.