Skip to content
Snippets Groups Projects
  1. Jul 17, 2020
  2. Jul 16, 2020
  3. Jul 15, 2020
  4. Jul 14, 2020
    • Alexandre Janniaux's avatar
      test: media_player: check multiple instance retain · 82e3111d
      Alexandre Janniaux authored
      Non-regression test for 12fa8556:
      
          lib: media_player: keep player libvlc instance
      82e3111d
    • Alexandre Janniaux's avatar
      lib: media_player: keep player libvlc instance · 12fa8556
      Alexandre Janniaux authored
      The media_player instance retains and releases the libvlc instance it's
      created from. When using a different libvlc instance for media_player
      and media, it was using the libvlc instance from the media_t object,
      leading to the release of the wrong libvlc instance, and thus potential
      use-after-free of one instance and leaks of the other.
      
      It has been spotted since VLCKit creates a shared libvlc instance and
      then can create a new libvlc instance in case the VLCMediaPlayer is
      created with different options, which means that the VLCMediaPlayer and
      the VLCMedia will be bound to different libvlc instances, triggering the
      issue described in first paragraph and crashing.
      
      /!\ This commit changes libvlc behaviour in the following case:
      
       - There's at least two libvlc instance, one being the parent of a media
         and the other being the parent of a media_player.
      
       - The media is set on the media_player.
      
       - A new media is played by the underlying vlc_player_t, in which case
         the new libvlc_media_t instance is now created with the player
         libvlc instance instead of the media libvlc instance.
      
       - The user call libvlc_media_player_get_media() on the player.
      
       - The user call preparse functions on the media.
      
      Since this very case was leading to crash anyway, and is quite
      intricate, it's probably not a very important change though.
      
      Refs videolan/VLCKit#189, videolan/VLCKit#116
      12fa8556
    • Alexandre Janniaux's avatar
      test: player: use tdummy text-renderer · 2d7a0bf4
      Alexandre Janniaux authored
      Avoid leaks in freetype when not testing it.
      2d7a0bf4
    • Alexandre Janniaux's avatar
      test: use adummy and tdummy by default in tests · 22f56c54
      Alexandre Janniaux authored
      Avoid leaks in pulse and freetype when not testing those.
      22f56c54
    • Alexandre Janniaux's avatar
      extras: ci: add apple tvos arm64 job · 9efab032
      Alexandre Janniaux authored
      Test compilation for apple tvOS, which has the particularity to need
      apple bitcode when compiling in order to ship it.
      9efab032
    • Alexandre Janniaux's avatar
      extras: ci: add iphonesimulator x86_64 job · b07d1a3f
      Alexandre Janniaux authored
      Since the iphonesimulator build has been broken multiple time because of
      cross-compilation miss-detection, add a job with same architecture as
      the CI. It builds with iphonesimulator, which could also be useful for
      running tests in the future.
      b07d1a3f
    • Alexandre Janniaux's avatar
      extras: ci: add SDKNAME for iOS jobs · b8975a20
      Alexandre Janniaux authored
      Generalize iOS jobs by supplying the SDK name as variable.
      
      It is mainly designed so as to have other SDK than iphoneos one, like
      iPhone simulator or Apple TV.
      b8975a20
  5. Jul 13, 2020
Loading