[vlc-devel] commit "gnutls: use vectorized sends on all platforms" breaks Android

[Jean-Baptiste Kempf]

On 18 Jan, Rémi Denis-Courmont wrote :
> Le 2016-01-18 16:22, Andrey Gursky a écrit :
> >On Mon, 18 Jan 2016 11:01:51 +0200
> >Rémi Denis-Courmont <remi at remlab.net> wrote:
> >
> >>Le 2016-01-18 03:57, Andrey Gursky a écrit :
> >>> Hi,
> >>>
> >>> was it not a little bit to optimistic to remove #ifdef IOV_MAX?
> >>
> >>It was there to protect the sendmsg() call, as sendmsg() was missing on
> >>Windows. We have a replacement now.
> >
> >Please believe me, I wouldn't complain about it if it wouldn't break a
> >build on Android.
> 
> Heh, I wrote that ifdef and the code within it. I know why I added that
> ifdef (because Windows lacked sendmsg()), as I do why I removed it (because
> we have a replacement for sendmsg(() now).
> 
> IOV_MAX is *not* an optional fancy part of sendmsg(), or more generally, I/O
> vectors support. IOV_MAX exists to prevent overflows when copying or
> converting the vectors. VLC can't do without it, and the Linux kernel
> couldn't do without it either.

It seems that one needs to call sysconf(_SC_IOV_MAX) if IOV_MAX is not
defined in limits.h (XSI extension).

Not sure if Android is wrong there.

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device