cron2 has uploaded a new patch set (#5) to the change originally created by
stipa. ( http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
The following approvals got outdated and were removed:
Code-Review+2 by ordex
Change subject: ssl_openssl.c: Prevent potential double-free
.
Not tested, as the offending code flow is quite obvious "in hindsight", and
making the pointer NULL after free() is a sufficient safeguard against
double free(). A slightly more readable construction could have used
to different "info" variables for the two different scopes, each with
its own indi
From: Lev Stipakov
Fixes a potential double-free issue in tls_ctx_load_cert_uri()
by explicitly nullifying the pointer immediately after calling
OSSL_STORE_INFO_free(info).
This ensures that subsequent cleanup won't attempt to free the same pointer
again.
GitHub: #726
Change-Id: I4507be07cd5
Attention is currently required from: flichtenheld, plaisthos, stipa.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, plaisthos.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
..
P
Attention is currently required from: flichtenheld, ordex, plaisthos.
stipa has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, ordex, plaisthos, stipa.
Hello flichtenheld, ordex, plaisthos,
I'd like you to reexamine a change. Please visit
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email
to look at the new patch set (#4).
The following approvals got outdated an
Attention is currently required from: flichtenheld, plaisthos, stipa.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, ordex, plaisthos, stipa.
Hello flichtenheld, ordex, plaisthos,
I'd like you to reexamine a change. Please visit
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email
to look at the new patch set (#3).
Change subject: ssl_openssl.c: Prevent
Attention is currently required from: flichtenheld, ordex, plaisthos, stipa.
Hello flichtenheld, ordex, plaisthos,
I'd like you to reexamine a change. Please visit
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email
to look at the new patch set (#2).
The following approvals got outdated an
Attention is currently required from: flichtenheld, ordex, plaisthos.
stipa has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, plaisthos.
Hello plaisthos, flichtenheld,
I'd like you to do a code review.
Please visit
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email
to review the following change.
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, plaisthos, stipa.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/942?usp=email )
Change subject: ssl_openssl.c: Prevent potential double-free
...
Attention is currently required from: flichtenheld, its_Giaan, plaisthos.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/941?usp=email )
Change subject: Add support to "ovpn" aka "in upstream kernel"
.
Attention is currently required from: flichtenheld, its_Giaan, plaisthos.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/941?usp=email )
Change subject: Add support to "ovpn" aka "in upstream kernel"
.
Attention is currently required from: flichtenheld, its_Giaan, plaisthos.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/941?usp=email )
Change subject: Add support to "ovpn" aka "in upstream kernel"
.
Attention is currently required from: flichtenheld, its_Giaan, plaisthos.
ordex has posted comments on this change. (
http://gerrit.openvpn.net/c/openvpn/+/941?usp=email )
Change subject: Add support to "ovpn" aka "in upstream kernel"
.
17 matches
Mail list logo