[vlc-devel] [PATCH 0/3] access/satip: minor fixes

Filip Roséen filip at atch.se
Thu Mar 2 19:37:34 CET 2017


These patches are a follow-up to the discussion between me and Rémi
Denis-Cournmount linked below:

 - https://mailman.videolan.org/pipermail/vlc-devel/2017-March/111861.html

The changes are as minor as possible in order to not disturb the delicate
entity which the satip accessor is. One might for example ask why the
return-value of `vlc_UrlParse` is not checked, instead of checking the
individual attributes by themselves; and the reason is.. satip.

To quote Jean-Baptist Kempf:

    19:00:51 j-b: be EXTRA careful
    19:01:27 j-b: Satip is not conforming to any spec
    19:01:31 j-b: including URL spec

The changes in this patch-batch are as minor as impossible in order to not
break any existing usage of the module. The changes goal is to make the module
more robust, and prevent it from crashing on unexpected data, while not
disturbing it too much (perhaps leading to regressions).

Filip Roséen (3):
  access/satip: fix null-dereference on missing host
  access/satip: fix null-dereference on "malformed" protocol
  access/satip: check return-value of vlc_uri_compose

 modules/access/satip.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

-- 
2.12.0



More information about the vlc-devel mailing list