On 8/8/2021 9:28 AM, Michael Niedermayer wrote:
> +$req_slot =~ s/[^-._A-Za-z0-9 ]*//g;

I don't think it is wise to allow periods here, considering
we use string ops to append it to a dir name just below.

.. and we probably should not be using string ops to construct
directory names.

> +$req_time =~ s/[^0-9]*//g;

We should validate this is an actual time, probably.

- Derek
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to