[BUG] HEVC+AAC_LC and H264+EAC3 videos don't play after patching about videostation-ffmpeg-patcher HOT 17 CLOSED

did you played rhe video (and kept it playing in the background) when running the issue-report ?

@AlexPresso yes, but it just keeps loading, showing the loading spinner.

Are these videos playing without the patch ?

Here's the result with and without the patch:

As for checking the patch_utils.sh script, here you go:

root@:~#     ls -al /var/packages/VideoStation/patch
total 16
drwxr-xr-x 2 root root 4096 Dec 10 08:53 .
drwxr-xr-x 6 root root 4096 Dec 10 08:55 ..
-rw-r--r-- 1 root root   62 Dec 10 08:53 patch_config.sh
-rw-r--r-- 1 root root 1497 Dec 10 08:53 patch_utils.sh
root@:~#     ls -al /var/packages/CodecPack/patch
total 16
drwxr-xr-x 2 root root 4096 Dec 10 08:53 .
drwxr-xr-x 5 root root 4096 Dec 10 08:54 ..
-rw-r--r-- 1 root root   62 Dec 10 08:53 patch_config.sh
-rw-r--r-- 1 root root 1497 Dec 10 08:53 patch_utils.sh

Thank you for your time and help with this 🙇

from videostation-ffmpeg-patcher.

Thank you for testing, that's great news we're making progress and I know how to fix it, I'll make some change in the patcher to give right permissions to the scripts, I'll get back to you when everything ready :)

from videostation-ffmpeg-patcher.

Hi @pedroms

I don't see any issue or sign that ffmpeg was running from the issue-report, did you played rhe video (and kept it playing in the background) when running the issue-report ?

Are these videos playing without the patch ?

from videostation-ffmpeg-patcher.

I just seen that the patch_utils.sh script was not loaded by the ffmpeg wrapper, it's required to run:

FFMPEG prev logs...................................
Cannot load patch_utils.sh

Could you please give me the output of the following commands:

• ls -al /var/packages/VideoStation/patch
• ls -al /var/packages/CodecPack/patch

from videostation-ffmpeg-patcher.

Hi @pedroms, thanks to you for the feedback :) .

I don't have this issue using DSM 7.0+ but it could be a permission issue on DMS 6.2, could you please try to run the following commands to change the permissions of the patch files in VideoStation and CodecPack:

VideoStation

chown -R root:VideoStation /var/packages/VideoStation/patch
chmod -R 755 /var/packages/VideoStation/patch

CodecPack:

chmod -R 755 /var/packages/CodecPack/patch

After changing the permissions, please try to play a video file again and after waiting maximum 1 min, if nothing plays, please run the issue-report tool again and copy paste the output; so we can check if something changed or not :)

from videostation-ffmpeg-patcher.

@AlexPresso after updating the permissions with those commands, this is the result:

root@:~#     ls -al /var/packages/VideoStation/patch
total 16
drwxr-xr-x 2 root VideoStation 4096 Dec 15 09:35 .
drwxr-xr-x 6 root root         4096 Dec 15 09:45 ..
-rwxr-xr-x 1 root VideoStation   62 Dec 15 09:35 patch_config.sh
-rwxr-xr-x 1 root VideoStation 1497 Dec 15 09:35 patch_utils.sh
root@:~#     ls -al /var/packages/CodecPack/patch
total 16
drwxr-xr-x 2 root root 4096 Dec 15 09:35 .
drwxr-xr-x 5 root root 4096 Dec 15 09:36 ..
-rwxr-xr-x 1 root root   62 Dec 15 09:35 patch_config.sh
-rwxr-xr-x 1 root root 1497 Dec 15 09:35 patch_utils.sh

I tried playing the video again (mp4, hevc, aac_lc) and it did not work 🙁 Here's the report: https://gist.github.com/pedroms/59173b5eb2eb10da2b14d2a58517e2d5

from videostation-ffmpeg-patcher.

@pedroms if you run the following command in your terminal, do you have any error ? source /var/packages/VideoStation/patch/patch_utils.sh ?

from videostation-ffmpeg-patcher.

@AlexPresso no error, it does not return anything:

root@:~# source /var/packages/VideoStation/patch/patch_utils.sh
root@:~# 

from videostation-ffmpeg-patcher.

Hi @pedroms

Thank you for the tests, I added some error logging in the following PR (#116) it's not a fix for your issue yet, just a step for me to try get more informations about your issue, would it be possible for you to try it please:

• uninstall the patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/more_error_logging/patcher.sh | bash -s -- -a unpatch
• install the error logging patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/more_error_logging/patcher.sh | bash -s -- -a patch -b more_error_logging

Please try to run another video file and run the issue-report script again, hopefully we can get the error thrown when the script is trying to load patch_utils.sh

from videostation-ffmpeg-patcher.

@AlexPresso followed your instructions. Here's the latest issue report: https://gist.github.com/pedroms/01f2e056d9557abdc7e26bc091c284d8

from videostation-ffmpeg-patcher.

Could you just do this one last try before I make changes to the patcher please:
chmod -R 755 /var/packages/CodecPack/target/pack/bin

And try to play a video file as is

from videostation-ffmpeg-patcher.

I pushed a new test wrapper in #116

You can try it out using the following:

• uninstall the patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/more_error_logging/patcher.sh | bash -s -- -a unpatch
• install the test patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/more_error_logging/patcher.sh | bash -s -- -a patch -b more_error_logging

I'm not facing the same issue on DSM 7.0+, so it's possible the setuid on the CodecPack's scripts is messing with DSM 6.2 apparmor configuration, let's try :)

from videostation-ffmpeg-patcher.

@AlexPresso thank you for your work to fix this. I followed your latest instructions and, even with the new test patch, the video fails to play 😕 Similar to the previous times, I ran the report tool while the video is trying to play: https://gist.github.com/pedroms/b5c18c64d22c0300a085fabebe4f66ce

from videostation-ffmpeg-patcher.

Okay, I'll remove these utils script and put everything back in the ffmpeg wrapper directly, so the script won't need to load any external file again

Sorry, it took long time but I wanted to test everything before choosing this solution

from videostation-ffmpeg-patcher.

Hi @pedroms

I pushed a new test wrapper in #117

You can try it out using the following:

• uninstall the old patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/main/patcher.sh | bash -s -- -a unpatch
• install the test patch : curl https://raw.githubusercontent.com/AlexPresso/VideoStation-FFMPEG-Patcher/rem_extern_scripts/patcher.sh | bash -s -- -a patch -b rem_extern_scripts

There are no more external scripts to load so there should not be any permission issue anymore

from videostation-ffmpeg-patcher.

@AlexPresso 🎉 it's all fixed now! Here's the result with the test patch:

Thank you so much for your help and for keeping this patcher alive and well. Happy holidays!

from videostation-ffmpeg-patcher.

You're welcome, thanks to you for testing and for your patience :)

Happy Holidays ! 🎅

from videostation-ffmpeg-patcher.