On 18/07/2019 14:20, Lev Stipakov wrote:
> Hi,
>
>
>> Are we sure we can't call this function with 'len' longer than the
>> available space in 'key' ?
>
>
> Yep, because we get available space here:
>
>> +const int len = BCAP(key);
>>
>
> and then pass it to rand_bytes:
>
Ok, that so
Hi,
> Are we sure we can't call this function with 'len' longer than the
> available space in 'key' ?
Yep, because we get available space here:
> +const int len = BCAP(key);
>
and then pass it to rand_bytes:
> > +if (!rand_bytes(BEND(key), len))
--
-Lev
__
Hi,
On 18/07/2019 11:35, Lev Stipakov wrote:
> From: Lev Stipakov
>
> Commit fb4e8ab added variable-length array which
> is C99 feature and is not supported by Visual Studio.
>
> This removes VLA and writes data directly into passed buffer.
>
> Signed-off-by: Lev Stipakov
> ---
> src/openvpn
From: Lev Stipakov
This patch enables interactive service to open tun device.
This is mostly needed by Wintun, which could be opened
only by privileged process.
When interactive service is used, instead of calling
CreateFile() directly by openvpn process we pass tun device path
into service proc
From: Lev Stipakov
Commit fb4e8ab added variable-length array which
is C99 feature and is not supported by Visual Studio.
This removes VLA and writes data directly into passed buffer.
Signed-off-by: Lev Stipakov
---
src/openvpn/crypto.c | 10 +++---
1 file changed, 7 insertions(+), 3 dele
Hi,
> That'll probably work with some extra sanity checks on the file name.
> Ideally we should just pass the dev-node (empty if unspecified) and type of
> device (TAP6 or WINTUN), but that will require a lot of duplication of
> code in the service, as you noted.
>
> One option is to pass the de
