[vlc-devel] [PATCH] stream_ReadLine: increase line length limit

Francois Cartegnie fcvlcdev at free.fr
Wed Jul 29 17:07:09 CEST 2020


Le 20/07/2020 à 13:11, Pierre Ynard via vlc-devel a écrit :
> 
> Very long lines are occasionally encountered in text web resources such
> as HTML, JSON or other API data. This bumps the hardcoded limit from
> 200 kB to 4 MB, which should be sufficient to support more than most of
> them, but still reasonable to prevent any issue.

about that main topic,

There also can be no newline at all.

vlclua_stream_readline for those uses should be replaced by proper
stream format/node parsers with callbacks.
We already have json code & xml, that will imply new lua mappings and
rewriting the call code.

A low cost option is not to use Readline at all but read as binary and
tokenize in lua.

-- 
Francois Cartegnie
VideoLAN - VLC Developer


More information about the vlc-devel mailing list