diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/libcrypto/man/OPENSSL_sk_new.3 | 16 | ||||
| -rw-r--r-- | src/lib/libcrypto/stack/stack.c | 3 |
2 files changed, 6 insertions, 13 deletions
diff --git a/src/lib/libcrypto/man/OPENSSL_sk_new.3 b/src/lib/libcrypto/man/OPENSSL_sk_new.3 index 748411bf15..47b19cb4e8 100644 --- a/src/lib/libcrypto/man/OPENSSL_sk_new.3 +++ b/src/lib/libcrypto/man/OPENSSL_sk_new.3 | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | .\" $OpenBSD: OPENSSL_sk_new.3,v 1.8 2018/03/27 17:35:50 schwarze Exp $ | 1 | .\" $OpenBSD: OPENSSL_sk_new.3,v 1.9 2018/04/01 00:36:28 schwarze Exp $ |
| 2 | .\" | 2 | .\" |
| 3 | .\" Copyright (c) 2018 Ingo Schwarze <schwarze@openbsd.org> | 3 | .\" Copyright (c) 2018 Ingo Schwarze <schwarze@openbsd.org> |
| 4 | .\" | 4 | .\" |
| @@ -14,7 +14,7 @@ | |||
| 14 | .\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF | 14 | .\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF |
| 15 | .\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. | 15 | .\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. |
| 16 | .\" | 16 | .\" |
| 17 | .Dd $Mdocdate: March 27 2018 $ | 17 | .Dd $Mdocdate: April 1 2018 $ |
| 18 | .Dt OPENSSL_SK_NEW 3 | 18 | .Dt OPENSSL_SK_NEW 3 |
| 19 | .Os | 19 | .Os |
| 20 | .Sh NAME | 20 | .Sh NAME |
| @@ -279,8 +279,9 @@ or | |||
| 279 | successfuly calling | 279 | successfuly calling |
| 280 | .Fn sk_push , | 280 | .Fn sk_push , |
| 281 | .Fn sk_unshift , | 281 | .Fn sk_unshift , |
| 282 | or | ||
| 283 | .Fn sk_insert , | 282 | .Fn sk_insert , |
| 283 | or | ||
| 284 | .Fn sk_set , | ||
| 284 | or changing the comparison function sets the state to unsorted. | 285 | or changing the comparison function sets the state to unsorted. |
| 285 | If a comparison function is installed, calling | 286 | If a comparison function is installed, calling |
| 286 | .Fn sk_sort , | 287 | .Fn sk_sort , |
| @@ -564,15 +565,6 @@ first appeared in OpenSSL 0.9.7e and has been available since | |||
| 564 | first appeared in OpenSSL 0.9.8 and has been available since | 565 | first appeared in OpenSSL 0.9.8 and has been available since |
| 565 | .Ox 4.5 . | 566 | .Ox 4.5 . |
| 566 | .Sh BUGS | 567 | .Sh BUGS |
| 567 | .Fn sk_set | ||
| 568 | does not set the state of the | ||
| 569 | .Fa stack | ||
| 570 | to unsorted. | ||
| 571 | This can cause wrong results from subsequent | ||
| 572 | .Fn sk_find | ||
| 573 | and | ||
| 574 | .Fn sk_find_ex . | ||
| 575 | .Pp | ||
| 576 | Even if a comparison function is installed, empty stacks and | 568 | Even if a comparison function is installed, empty stacks and |
| 577 | stacks containing a single pointer are sometimes considered | 569 | stacks containing a single pointer are sometimes considered |
| 578 | sorted and sometimes considered unsorted. | 570 | sorted and sometimes considered unsorted. |
diff --git a/src/lib/libcrypto/stack/stack.c b/src/lib/libcrypto/stack/stack.c index d941f9e6fb..b76a0d7271 100644 --- a/src/lib/libcrypto/stack/stack.c +++ b/src/lib/libcrypto/stack/stack.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: stack.c,v 1.19 2015/02/07 13:19:15 doug Exp $ */ | 1 | /* $OpenBSD: stack.c,v 1.20 2018/04/01 00:36:28 schwarze Exp $ */ |
| 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * | 4 | * |
| @@ -312,6 +312,7 @@ sk_set(_STACK *st, int i, void *value) | |||
| 312 | { | 312 | { |
| 313 | if (!st || (i < 0) || (i >= st->num)) | 313 | if (!st || (i < 0) || (i >= st->num)) |
| 314 | return NULL; | 314 | return NULL; |
| 315 | st->sorted = 0; | ||
| 315 | return (st->data[i] = value); | 316 | return (st->data[i] = value); |
| 316 | } | 317 | } |
| 317 | 318 | ||
