Mehdi Sabwat authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

disable a52, aom, faad2, chromaprint, mad, libmpeg2, nvcodec, tremov, vpx and theora.

refs jbk/vlc.js#38

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

The implementation of the MTP service discovery could only support one
device at a time.

Now that the structures to store a list of devices is in place, perform
a diff between the known devices and the detected devices to update the
list accordingly.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

Refactor to use a list of devices, and use a device node, parent of all
discovered media for the device.

This also prepares to add support for several MTP devices at the same
time.

Fixes #26085

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

Allocating the pp_items array required to know the number of items in
advance.

To do so, the mtp module hijacked the progression callback to know the
number of items on listing (which was called once for each item, but in
the end the total value was correct).

This callback function accepts a "userdata" having type `void const *`,
so it not intended to write to it, but it still worked without ugly cast
due to an additional redirection (p_sys).

To prepare further refactors (multi-device support), remove this
callback hack and use a vlc_vector to append items on-the-fly.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

Do not leak the remaining items.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

No functional changes.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

LIBMTP_Get_Friendlyname() may return an empty string. Since it is not
NULL, this name was used as the device name.

Instead, in that case, fallback to the model name.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

When pl_scale could not be loaded, explicitly log an error to report
that --gl-upscaler and --gl-downscaler options will be ignored.

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

The upscaler and downscaler options are valid for all OpenGL vouts.
The fact that they use libplacebo is an implementation detail (they
could support another backend later).

In practice, this also avoids to crash if libplacebo is disabled:
the variables were defined only if HAVE_LIBPLACEBO, but they were read
in all cases.

Fixes #26315

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

HAVE_LIBPLACEBO_SCALE checks the version of libplacebo, but if
libplacebo is explicitly disabled (but libplacebo_scale enabled), then
it failed.

Pierre Ynard authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

They now appear; add support for parsing them.

Fixes #26317

Refs #26296.

Steve Lhomme authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

Negative or VLC_TICK_INVALID timestamps are not sent as a PCR but there might
still be such blocks sent before the PCR.

fix #26309

Romain Vimont authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

Commit 47a00338 added the wrapper
function, but did not call it.

Alexandre Janniaux authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

Each cuInit is leaking memory, and cuInit is supposed to be called only
once. Ensure it through a vlc_once_t wrapper, but since the function
table needs to be loaded in order to call into cuda functions, forward
some available device for the initialization.

Below, there is two leaks because of the double decoder device
allocation in debug mode. The first leak is leaked once, and is still
there. The second and third leaks are now reduced to a single one.

Direct leak of 65536 byte(s) in 1 object(s) allocated from:
    #0 0x7febb6bf3652 in __interceptor_realloc /build/gcc/src/gcc/libsanitizer/asan/asan_malloc_linux.cpp:164
    #1 0x7feb8b56828f  (<unknown module>)

Direct leak of 56 byte(s) in 1 object(s) allocated from:
    #0 0x7febb6bf3459 in __interceptor_calloc /build/gcc/src/gcc/libsanitizer/asan/asan_malloc_linux.cpp:154
    #1 0x7feb89dc586a  (/home/alexandre/workspace/videolabs/vlc-meson/build-native/modules/.libs/libskins2_plugin.so+0x2f1086a)
    #2 0x7febb5f195c1 in decoder_device_Open ../../src/input/decoder_helpers.c:174
    #3 0x7febb5e67106 in vlc_module_load ../../src/modules/modules.c:243
    #4 0x7febb5f196e8 in vlc_decoder_device_Create ../../src/input/decoder_helpers.c:188
    #5 0x7febb60a7435 in vout_GetDevice ../../src/video_output/video_output.c:2244
    #6 0x7febb5ef7f26 in ModuleThread_GetDecoderDevice ../../src/input/decoder.c:608
    #7 0x7feba02ee139 in decoder_GetDecoderDevice ../../include/vlc_codec.h:304
    #8 0x7feba02fb51e in lavc_UpdateVideoFormat ../../modules/codec/avcodec/video.c:286
    #9 0x7feba0313ff7 in ffmpeg_GetFormat ../../modules/codec/avcodec/video.c:1682
    #10 0x7feb9f0bfe12  (/usr/lib/libavcodec.so.58+0x29ae12)

Direct leak of 56 byte(s) in 1 object(s) allocated from:
    #0 0x7febb6bf3459 in __interceptor_calloc /build/gcc/src/gcc/libsanitizer/asan/asan_malloc_linux.cpp:154
    #1 0x7feb8b5d386a  (<unknown module>)
    #2 0x7febb5f195c1 in decoder_device_Open ../../src/input/decoder_helpers.c:174
    #3 0x7febb5e67106 in vlc_module_load ../../src/modules/modules.c:243
    #4 0x7febb5f196e8 in vlc_decoder_device_Create ../../src/input/decoder_helpers.c:188
    #5 0x7febb60a7435 in vout_GetDevice ../../src/video_output/video_output.c:2244
    #6 0x7febb5ef7f26 in ModuleThread_GetDecoderDevice ../../src/input/decoder.c:608
    #7 0x7feba02ee139 in decoder_GetDecoderDevice ../../include/vlc_codec.h:304
    #8 0x7feba02fb51e in lavc_UpdateVideoFormat ../../modules/codec/avcodec/video.c:286
    #9 0x7feba0313ff7 in ffmpeg_GetFormat ../../modules/codec/avcodec/video.c:1682
    #10 0x7feb9f0bfe12  (/usr/lib/libavcodec.so.58+0x29ae12)