- fix the logic
- simplify
- repair the ability to save the state and position
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Close #5699
Regression from 43424ac2

Closes #5707 #5493

The error message  "EXT-X-MEDIA-SEQUENCE already present in playlist (new=%d, old=%d)" was
triggered way to often for HLS live streams. In addition the new metadata from a new HLS
playlist was not updated in the internal list.

Close #5700
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Using it causes libdvdnav to complain about 'demux error'.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Close #5704
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

this commit also fixes a xib incompatiblity with 10.5

macosx: correctly hide the controls within the native fullscreen mode on Lion when displaying a video (closes #5505)

note that the ordinary fullscreen controller panel won't be present atm though

additionally, this commit can potentially impact the layout of the controller zone and lead to ugly effects, so please stress it a bit

macosx: fixed missing menubar on Lion when entering fullscreen mode through a VLC internal trigger (fixes #5537)

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Oops, Do not forget to include limits.h when using INT_MAX

Use INT_MAX as the maximum content length as libupnp introduced 0 as an
unlimited content length in pupnp 1.6.7, though UPNP_VERSION was not bumped.

These buffers aren't known to the actual OMX codec.

Normally, these buffers are removed from the queue by
DecodeAudio/DecodeVideo, but if they're pushed multiple
times (which happens when using google sw decoders),
PortReconfigure might try to free them.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

These buffers were introduced to avoid a potential deadlock
when the codec has issued a PortSettingsChanged event but
not returned any buffers to the client, making the DecodeVideo
function hang waiting for an input buffer to use, before
handling the reconfigure. The same handling needs to be done
for DecodeAudio too, otherwise it will try to use the
sentinel buffer as a normal one.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

This SetParameters call shouldn't be necessary as far as I know,
but only avoiding it for audio codecs for now.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

This is similar to the ThumbnailMode on Nexus S, except that
the index doesn't seem to be queryable dynamically
(I don't seem to find any string in the binaries that
would be the right one at least).
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Fixes #5640
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Setting libupnp's maximum content length to zero makes it unable to
fetch any didl instead of the probably expected unlimited behavior.

* Accept any version of the MediaServer service
* Browse the advertised ContentDirectory service version
Signed-off-by: Mirsal Ennaime <mirsal@videolan.org>

$enable_speex can't be empty at this point so this check makes no sense

According to http://upnp.org/specs/av/UPnP-av-MediaServer-v1-Device.pdf, the tag name of the element
that provides the base URL for relative URLs is "URLBase". The same is true for a v2 MediaServer.
Make it so in MediaServer:: parseDeviceDescription().
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Close #5675

Because ExecuteCommand allocates temporary string space on the stack,
proportional to the length of the command to execute, a stack overflow can
occur when the the command is too long.

This can be triggered remotely e.g. from the VLC Web interface, by
running this JavaScript:

sendVLMCmd(Array.prototype.join.call({length:300000},'a'));

which sends a string of length 300000 to ExecuteCommand, crashing VLC.
OKed-by: Rafaël Carré <funman@videolan.org>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

The iomx loading stub doesn't use the library name.
If there's errors in loading the function pointers, this
gives slightly more correct error messages.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>