Christoph Anton Mitterer
Christoph Anton Mitterer
Last but not least, overwriting. AFAIU `io.open(chapters_file_path, "w")` overwrites any file... AFAICT it's safe though, because you use `mpv`’s `path` and `"filename/no-ext"` + extension... so that shouln't be abusable to...
I just tried the case at: https://github.com/mar04/chapters_for_mpv/blob/78a8513e417daef4aff1a8fb2bba20a804167a18/chapters.lua#L322-L334 Create for example a video file named `'foo; touch PWNED .webm'` and enable `global_chapters`. Even without any chapters already created, just by playing...
PR #5 should fix at least the thing with the code injection and the possibility of taking a pathname for a command option. Leaves us with possible path traversals, possibly...
`fritzconnection` has now been officially [packaged](https://packages.debian.org/sid/python3-fritzconnection) for Debian, doesn't directly help you for `bookworm`, but at least for future versions (and in principle you could of course pull it into...
also happens on Linux (where IMO the options should just be ignored)
I've just submitted #3131, but that only fixes `--binary` on **non**-windows (and at least form the code, I couldn't see that it was broken in Windows). Same for `--nul-output` (which...
`--nul-output` is now `--raw-output0` So do you think this issue can be closed?
Just for the records: There's has also been something coded in `needrestart` itself: https://github.com/liske/needrestart/issues/291
As I [found out after some debugging](https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1091825#24) (with having literally no clue about ffmpeg development ^^), at least the problem I’ve seen and described in the aforementioned is probably already...
@lalinsky Could you perhaps release a new version with the few commits since 1.5.1?