[vlc-devel] [PATCH 09/12] network: io: Remove VLA usages

Pierre Ynard linkfanel at yahoo.fr
Thu Dec 10 12:10:14 CET 2020


> Using VLA for this particular part of the code (http_auth) is very
> convenient since the size max is bounded, so it can't overflow.
> 
> Having said that, we are very close to a full support of MSVC. If
> we need to fix some well-written code to remove VLA, I'm OK for it.
> I'm convinced by Steve's arguments. Supporting MSVC can bring more
> developers in VLC.

Can we try using some #ifdef and/or converting VLAs to FLAs, rather than
to heap allocation, where the size is reasonably bounded?

> Should we vote for VLA removal in VLC ? Via the TC ?

That seems a bit too forward-leaning to me, this is only the first
version of the patchset.

-- 
Pierre Ynard
"Une âme dans un corps, c'est comme un dessin sur une feuille de papier."


More information about the vlc-devel mailing list