[FFmpeg-devel] [PATCH v3 1/2] avformat/url: fix logic for removing ".." path components

[Nicolas George]

Marton Balint (12020-07-29):
> Thanks for working on this. I agree that proper (as RFC compliant as it can
> be) URL parsing is needed here. Probably we should clearly document
> differences from RFC compliant parsing, if we cannot do it entirely RFC
> compliantly...

Indeed. Unfortunately, that may cause some trouble, but not for right
now.

> ? and # can also separate authority from the rest.

Thanks, fixed.

> I am not sure about this approach, the known characters at the end or at the
> start will make further operations a bit harder. I'd just simply add another
> field for each URL component to signal the end, e.g.

I considered this, but it was not actually convenient.

I have something that works, but it needs polishing and more testing.
But you can have a peek.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-WIP-lavf-url-rewrite-ff_make_absolute_url.patch
Type: text/x-diff
Size: 9027 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200730/bf2d973e/attachment.patch>