-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make SSL_clear_options pass new options to record layer #23045
Conversation
ssl/ssl_lib.c
Outdated
@@ -6023,6 +6023,7 @@ uint64_t SSL_set_options(SSL *s, uint64_t op) | |||
|
|||
/* Ignore return value */ | |||
sc->rlayer.rrlmethod->set_options(sc->rlayer.rrl, options); | |||
sc->rlayer.rrlmethod->set_options(sc->rlayer.wrl, options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be wrlmethod
here.
ssl/ssl_lib.c
Outdated
|
||
/* Ignore return value */ | ||
sc->rlayer.rrlmethod->set_options(sc->rlayer.rrl, options); | ||
sc->rlayer.rrlmethod->set_options(sc->rlayer.wrl, options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrlmethod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Signed-off-by: lan1120 <lanming@huawei.com>
Needs reapproval |
This pull request is ready to merge |
Pushed to master and 3.2. Thanks. |
Signed-off-by: lan1120 <lanming@huawei.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from #23045)
Signed-off-by: lan1120 <lanming@huawei.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from openssl/openssl#23045) Signed-off-by: lanming1120 <lanming1120@126.com>
Signed-off-by: lan1120 <lanming@huawei.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from openssl/openssl#23045) (cherry picked from commit e8e95f20a9b00ca62d407263110663eba7614683) Signed-off-by: lanming1120 <lanming1120@126.com>
Signed-off-by: lan1120 <lanming@huawei.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from openssl#23045)
Make SSL_clear_options pass new options to record layer, so we can more flexibly control certain features, such as DTLS record replay check.