Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Message Processing Stack, Reader implementation |
| 3 | * |
| 4 | * Copyright The Mbed TLS Contributors |
Dave Rodgman | 16799db | 2023-11-02 19:47:20 +0000 | [diff] [blame^] | 5 | * SPDX-License-Identifier: Apache-2.0 OR GPL-2.0-or-later |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 6 | * |
| 7 | * This file is part of Mbed TLS (https://tls.mbed.org) |
| 8 | */ |
| 9 | |
Hanno Becker | 43c8f8c | 2021-03-05 05:16:45 +0000 | [diff] [blame] | 10 | #include "common.h" |
| 11 | |
Ronald Cron | 6f135e1 | 2021-12-08 16:57:54 +0100 | [diff] [blame] | 12 | #if defined(MBEDTLS_SSL_PROTO_TLS1_3) |
Hanno Becker | 43c8f8c | 2021-03-05 05:16:45 +0000 | [diff] [blame] | 13 | |
Hanno Becker | c518c3b | 2021-01-28 07:08:08 +0000 | [diff] [blame] | 14 | #include "mps_reader.h" |
| 15 | #include "mps_common.h" |
| 16 | #include "mps_trace.h" |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 17 | |
| 18 | #include <string.h> |
| 19 | |
Hanno Becker | 984fbde | 2021-01-28 09:02:18 +0000 | [diff] [blame] | 20 | #if defined(MBEDTLS_MPS_ENABLE_TRACE) |
| 21 | static int mbedtls_mps_trace_id = MBEDTLS_MPS_TRACE_BIT_READER; |
| 22 | #endif /* MBEDTLS_MPS_ENABLE_TRACE */ |
Hanno Becker | b910016 | 2021-01-12 09:46:03 +0000 | [diff] [blame] | 23 | |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 24 | /* |
| 25 | * GENERAL NOTE ON CODING STYLE |
| 26 | * |
| 27 | * The following code intentionally separates memory loads |
| 28 | * and stores from other operations (arithmetic or branches). |
| 29 | * This leads to the introduction of many local variables |
| 30 | * and significantly increases the C-code line count, but |
| 31 | * should not increase the size of generated assembly. |
| 32 | * |
Hanno Becker | fea81b3 | 2021-02-22 15:18:11 +0000 | [diff] [blame] | 33 | * The reason for this is twofold: |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 34 | * (1) It will ease verification efforts using the VST |
Hanno Becker | fea81b3 | 2021-02-22 15:18:11 +0000 | [diff] [blame] | 35 | * (Verified Software Toolchain) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 36 | * whose program logic cannot directly reason |
| 37 | * about instructions containing a load or store in |
| 38 | * addition to other operations (e.g. *p = *q or |
| 39 | * tmp = *p + 42). |
| 40 | * (2) Operating on local variables and writing the results |
| 41 | * back to the target contexts on success only |
| 42 | * allows to maintain structure invariants even |
| 43 | * on failure - this in turn has two benefits: |
| 44 | * (2.a) If for some reason an error code is not caught |
| 45 | * and operation continues, functions are nonetheless |
| 46 | * called with sane contexts, reducing the risk |
| 47 | * of dangerous behavior. |
| 48 | * (2.b) Randomized testing is easier if structures |
| 49 | * remain intact even in the face of failing |
| 50 | * and/or non-sensical calls. |
| 51 | * Moreover, it might even reduce code-size because |
| 52 | * the compiler need not write back temporary results |
| 53 | * to memory in case of failure. |
| 54 | * |
| 55 | */ |
| 56 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 57 | static inline int mps_reader_is_accumulating( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 58 | mbedtls_mps_reader const *rd) |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 59 | { |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 60 | mbedtls_mps_size_t acc_remaining; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 61 | if (rd->acc == NULL) { |
| 62 | return 0; |
| 63 | } |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 64 | |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 65 | acc_remaining = rd->acc_share.acc_remaining; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 66 | return acc_remaining > 0; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 67 | } |
| 68 | |
| 69 | static inline int mps_reader_is_producing( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 70 | mbedtls_mps_reader const *rd) |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 71 | { |
| 72 | unsigned char *frag = rd->frag; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 73 | return frag == NULL; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 74 | } |
| 75 | |
| 76 | static inline int mps_reader_is_consuming( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 77 | mbedtls_mps_reader const *rd) |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 78 | { |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 79 | return !mps_reader_is_producing(rd); |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 80 | } |
| 81 | |
| 82 | static inline mbedtls_mps_size_t mps_reader_get_fragment_offset( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 83 | mbedtls_mps_reader const *rd) |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 84 | { |
| 85 | unsigned char *acc = rd->acc; |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 86 | mbedtls_mps_size_t frag_offset; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 87 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 88 | if (acc == NULL) { |
| 89 | return 0; |
| 90 | } |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 91 | |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 92 | frag_offset = rd->acc_share.frag_offset; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 93 | return frag_offset; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 94 | } |
| 95 | |
| 96 | static inline mbedtls_mps_size_t mps_reader_serving_from_accumulator( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 97 | mbedtls_mps_reader const *rd) |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 98 | { |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 99 | mbedtls_mps_size_t frag_offset, end; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 100 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 101 | frag_offset = mps_reader_get_fragment_offset(rd); |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 102 | end = rd->end; |
| 103 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 104 | return end < frag_offset; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 105 | } |
| 106 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 107 | static inline void mps_reader_zero(mbedtls_mps_reader *rd) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 108 | { |
| 109 | /* A plain memset() would likely be more efficient, |
| 110 | * but the current way of zeroing makes it harder |
| 111 | * to overlook fields which should not be zero-initialized. |
Hanno Becker | 0bea62f | 2021-02-08 07:54:19 +0000 | [diff] [blame] | 112 | * It's also more suitable for FV efforts since it |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 113 | * doesn't require reasoning about structs being |
| 114 | * interpreted as unstructured binary blobs. */ |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 115 | static mbedtls_mps_reader const zero = |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 116 | { .frag = NULL, |
| 117 | .frag_len = 0, |
| 118 | .commit = 0, |
| 119 | .end = 0, |
| 120 | .pending = 0, |
| 121 | .acc = NULL, |
| 122 | .acc_len = 0, |
| 123 | .acc_available = 0, |
| 124 | .acc_share = { .acc_remaining = 0 } }; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 125 | *rd = zero; |
| 126 | } |
| 127 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 128 | int mbedtls_mps_reader_init(mbedtls_mps_reader *rd, |
| 129 | unsigned char *acc, |
| 130 | mbedtls_mps_size_t acc_len) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 131 | { |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 132 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_init"); |
| 133 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 134 | "* Accumulator size: %u bytes", (unsigned) acc_len); |
| 135 | mps_reader_zero(rd); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 136 | rd->acc = acc; |
| 137 | rd->acc_len = acc_len; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 138 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 139 | } |
| 140 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 141 | int mbedtls_mps_reader_free(mbedtls_mps_reader *rd) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 142 | { |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 143 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_free"); |
| 144 | mps_reader_zero(rd); |
| 145 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 146 | } |
| 147 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 148 | int mbedtls_mps_reader_feed(mbedtls_mps_reader *rd, |
| 149 | unsigned char *new_frag, |
| 150 | mbedtls_mps_size_t new_frag_len) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 151 | { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 152 | mbedtls_mps_size_t copy_to_acc; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 153 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_feed"); |
| 154 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 155 | "* Fragment length: %u bytes", (unsigned) new_frag_len); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 156 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 157 | if (new_frag == NULL) { |
| 158 | MBEDTLS_MPS_TRACE_RETURN(MBEDTLS_ERR_MPS_READER_INVALID_ARG); |
| 159 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 160 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 161 | MBEDTLS_MPS_STATE_VALIDATE_RAW(mps_reader_is_producing( |
| 162 | rd), |
| 163 | "mbedtls_mps_reader_feed() requires reader to be in producing mode"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 164 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 165 | if (mps_reader_is_accumulating(rd)) { |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 166 | unsigned char *acc = rd->acc; |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 167 | mbedtls_mps_size_t acc_remaining = rd->acc_share.acc_remaining; |
| 168 | mbedtls_mps_size_t acc_available = rd->acc_available; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 169 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 170 | /* Skip over parts of the accumulator that have already been filled. */ |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 171 | acc += acc_available; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 172 | |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 173 | copy_to_acc = acc_remaining; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 174 | if (copy_to_acc > new_frag_len) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 175 | copy_to_acc = new_frag_len; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 176 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 177 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 178 | /* Copy new contents to accumulator. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 179 | memcpy(acc, new_frag, copy_to_acc); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 180 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 181 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 182 | "Copy new data of size %u of %u into accumulator at offset %u", |
| 183 | (unsigned) copy_to_acc, (unsigned) new_frag_len, |
| 184 | (unsigned) acc_available); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 185 | |
| 186 | /* Check if, with the new fragment, we have enough data. */ |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 187 | acc_remaining -= copy_to_acc; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 188 | if (acc_remaining > 0) { |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 189 | /* We need to accumulate more data. Stay in producing mode. */ |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 190 | acc_available += copy_to_acc; |
| 191 | rd->acc_share.acc_remaining = acc_remaining; |
| 192 | rd->acc_available = acc_available; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 193 | MBEDTLS_MPS_TRACE_RETURN(MBEDTLS_ERR_MPS_READER_NEED_MORE); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 194 | } |
| 195 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 196 | /* We have filled the accumulator: Move to consuming mode. */ |
| 197 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 198 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 199 | "Enough data available to serve user request"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 200 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 201 | /* Remember overlap of accumulator and fragment. */ |
Hanno Becker | 032b352 | 2021-03-08 16:23:26 +0000 | [diff] [blame] | 202 | rd->acc_share.frag_offset = acc_available; |
| 203 | acc_available += copy_to_acc; |
| 204 | rd->acc_available = acc_available; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 205 | } else { /* Not accumulating */ |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 206 | rd->acc_share.frag_offset = 0; |
| 207 | } |
| 208 | |
| 209 | rd->frag = new_frag; |
| 210 | rd->frag_len = new_frag_len; |
| 211 | rd->commit = 0; |
| 212 | rd->end = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 213 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 214 | } |
| 215 | |
| 216 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 217 | int mbedtls_mps_reader_get(mbedtls_mps_reader *rd, |
| 218 | mbedtls_mps_size_t desired, |
| 219 | unsigned char **buffer, |
| 220 | mbedtls_mps_size_t *buflen) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 221 | { |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 222 | unsigned char *frag; |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 223 | mbedtls_mps_size_t frag_len, frag_offset, end, frag_fetched, frag_remaining; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 224 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_get"); |
| 225 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 226 | "* Bytes requested: %u", (unsigned) desired); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 227 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 228 | MBEDTLS_MPS_STATE_VALIDATE_RAW(mps_reader_is_consuming( |
| 229 | rd), |
| 230 | "mbedtls_mps_reader_get() requires reader to be in consuming mode"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 231 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 232 | end = rd->end; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 233 | frag_offset = mps_reader_get_fragment_offset(rd); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 234 | |
| 235 | /* Check if we're still serving from the accumulator. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 236 | if (mps_reader_serving_from_accumulator(rd)) { |
Hanno Becker | 77e4f48 | 2021-02-22 16:46:06 +0000 | [diff] [blame] | 237 | /* Illustration of supported and unsupported cases: |
| 238 | * |
| 239 | * - Allowed #1 |
| 240 | * |
| 241 | * +-----------------------------------+ |
| 242 | * | frag | |
| 243 | * +-----------------------------------+ |
| 244 | * |
| 245 | * end end+desired |
| 246 | * | | |
| 247 | * +-----v-------v-------------+ |
| 248 | * | acc | |
| 249 | * +---------------------------+ |
| 250 | * | | |
Hanno Becker | ecb02fb | 2021-03-26 19:20:49 +0000 | [diff] [blame] | 251 | * frag_offset acc_available |
Hanno Becker | 77e4f48 | 2021-02-22 16:46:06 +0000 | [diff] [blame] | 252 | * |
| 253 | * - Allowed #2 |
| 254 | * |
| 255 | * +-----------------------------------+ |
| 256 | * | frag | |
| 257 | * +-----------------------------------+ |
| 258 | * |
| 259 | * end end+desired |
| 260 | * | | |
| 261 | * +----------v----------------v |
| 262 | * | acc | |
| 263 | * +---------------------------+ |
| 264 | * | | |
Hanno Becker | ecb02fb | 2021-03-26 19:20:49 +0000 | [diff] [blame] | 265 | * frag_offset acc_available |
Hanno Becker | 77e4f48 | 2021-02-22 16:46:06 +0000 | [diff] [blame] | 266 | * |
| 267 | * - Not allowed #1 (could be served, but we don't actually use it): |
| 268 | * |
| 269 | * +-----------------------------------+ |
| 270 | * | frag | |
| 271 | * +-----------------------------------+ |
| 272 | * |
| 273 | * end end+desired |
| 274 | * | | |
| 275 | * +------v-------------v------+ |
| 276 | * | acc | |
| 277 | * +---------------------------+ |
| 278 | * | | |
Hanno Becker | ecb02fb | 2021-03-26 19:20:49 +0000 | [diff] [blame] | 279 | * frag_offset acc_available |
Hanno Becker | 77e4f48 | 2021-02-22 16:46:06 +0000 | [diff] [blame] | 280 | * |
| 281 | * |
| 282 | * - Not allowed #2 (can't be served with a contiguous buffer): |
| 283 | * |
| 284 | * +-----------------------------------+ |
| 285 | * | frag | |
| 286 | * +-----------------------------------+ |
| 287 | * |
| 288 | * end end + desired |
| 289 | * | | |
| 290 | * +------v--------------------+ v |
| 291 | * | acc | |
| 292 | * +---------------------------+ |
| 293 | * | | |
Hanno Becker | ecb02fb | 2021-03-26 19:20:49 +0000 | [diff] [blame] | 294 | * frag_offset acc_available |
Hanno Becker | 77e4f48 | 2021-02-22 16:46:06 +0000 | [diff] [blame] | 295 | * |
| 296 | * In case of Allowed #2 we're switching to serve from |
| 297 | * `frag` starting from the next call to mbedtls_mps_reader_get(). |
| 298 | */ |
| 299 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 300 | unsigned char *acc; |
| 301 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 302 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 303 | "Serve the request from the accumulator"); |
| 304 | if (frag_offset - end < desired) { |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 305 | mbedtls_mps_size_t acc_available; |
| 306 | acc_available = rd->acc_available; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 307 | if (acc_available - end != desired) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 308 | /* It might be possible to serve some of these situations by |
| 309 | * making additional space in the accumulator, removing those |
| 310 | * parts that have already been committed. |
| 311 | * On the other hand, this brings additional complexity and |
| 312 | * enlarges the code size, while there doesn't seem to be a use |
| 313 | * case where we don't attempt exactly the same `get` calls when |
| 314 | * resuming on a reader than what we tried before pausing it. |
| 315 | * If we believe we adhere to this restricted usage throughout |
| 316 | * the library, this check is a good opportunity to |
| 317 | * validate this. */ |
Hanno Becker | 984fbde | 2021-01-28 09:02:18 +0000 | [diff] [blame] | 318 | MBEDTLS_MPS_TRACE_RETURN( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 319 | MBEDTLS_ERR_MPS_READER_INCONSISTENT_REQUESTS); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 320 | } |
| 321 | } |
| 322 | |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 323 | acc = rd->acc; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 324 | acc += end; |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 325 | |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 326 | *buffer = acc; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 327 | if (buflen != NULL) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 328 | *buflen = desired; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 329 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 330 | |
| 331 | end += desired; |
| 332 | rd->end = end; |
| 333 | rd->pending = 0; |
| 334 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 335 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | /* Attempt to serve the request from the current fragment */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 339 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 340 | "Serve the request from the current fragment."); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 341 | |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 342 | frag_len = rd->frag_len; |
| 343 | frag_fetched = end - frag_offset; /* The amount of data from the current |
| 344 | * fragment that has already been passed |
| 345 | * to the user. */ |
| 346 | frag_remaining = frag_len - frag_fetched; /* Remaining data in fragment */ |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 347 | |
| 348 | /* Check if we can serve the read request from the fragment. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 349 | if (frag_remaining < desired) { |
| 350 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 351 | "There's not enough data in the current fragment " |
| 352 | "to serve the request."); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 353 | /* There's not enough data in the current fragment, |
| 354 | * so either just RETURN what we have or fail. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 355 | if (buflen == NULL) { |
| 356 | if (frag_remaining > 0) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 357 | rd->pending = desired - frag_remaining; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 358 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 359 | "Remember to collect %u bytes before re-opening", |
| 360 | (unsigned) rd->pending); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 361 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 362 | MBEDTLS_MPS_TRACE_RETURN(MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 363 | } |
| 364 | |
| 365 | desired = frag_remaining; |
| 366 | } |
| 367 | |
| 368 | /* There's enough data in the current fragment to serve the |
| 369 | * (potentially modified) read request. */ |
Hanno Becker | f81e41f | 2021-02-08 08:04:01 +0000 | [diff] [blame] | 370 | |
| 371 | frag = rd->frag; |
| 372 | frag += frag_fetched; |
| 373 | |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 374 | *buffer = frag; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 375 | if (buflen != NULL) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 376 | *buflen = desired; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 377 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 378 | |
| 379 | end += desired; |
| 380 | rd->end = end; |
| 381 | rd->pending = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 382 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 383 | } |
| 384 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 385 | int mbedtls_mps_reader_commit(mbedtls_mps_reader *rd) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 386 | { |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 387 | mbedtls_mps_size_t end; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 388 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_commit"); |
| 389 | MBEDTLS_MPS_STATE_VALIDATE_RAW(mps_reader_is_consuming( |
| 390 | rd), |
| 391 | "mbedtls_mps_reader_commit() requires reader to be in consuming mode"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 392 | |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 393 | end = rd->end; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 394 | rd->commit = end; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 395 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 396 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 397 | } |
| 398 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 399 | int mbedtls_mps_reader_reclaim(mbedtls_mps_reader *rd, |
| 400 | int *paused) |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 401 | { |
| 402 | unsigned char *frag, *acc; |
| 403 | mbedtls_mps_size_t pending, commit; |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 404 | mbedtls_mps_size_t acc_len, frag_offset, frag_len; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 405 | MBEDTLS_MPS_TRACE_INIT("mbedtls_mps_reader_reclaim"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 406 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 407 | if (paused != NULL) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 408 | *paused = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 409 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 410 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 411 | MBEDTLS_MPS_STATE_VALIDATE_RAW(mps_reader_is_consuming( |
| 412 | rd), |
| 413 | "mbedtls_mps_reader_reclaim() requires reader to be in consuming mode"); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 414 | |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 415 | frag = rd->frag; |
| 416 | acc = rd->acc; |
| 417 | pending = rd->pending; |
| 418 | commit = rd->commit; |
| 419 | frag_len = rd->frag_len; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 420 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 421 | frag_offset = mps_reader_get_fragment_offset(rd); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 422 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 423 | if (pending == 0) { |
| 424 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 425 | "No unsatisfied read-request has been logged."); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 426 | |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 427 | /* Check if there's data left to be consumed. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 428 | if (commit < frag_offset || commit - frag_offset < frag_len) { |
| 429 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 430 | "There is data left to be consumed."); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 431 | rd->end = commit; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 432 | MBEDTLS_MPS_TRACE_RETURN(MBEDTLS_ERR_MPS_READER_DATA_LEFT); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 433 | } |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 434 | |
Hanno Becker | b185543 | 2021-02-08 08:07:35 +0000 | [diff] [blame] | 435 | rd->acc_available = 0; |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 436 | rd->acc_share.acc_remaining = 0; |
| 437 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 438 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 439 | "Fragment has been fully processed and committed."); |
| 440 | } else { |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 441 | int overflow; |
| 442 | |
| 443 | mbedtls_mps_size_t acc_backup_offset; |
| 444 | mbedtls_mps_size_t acc_backup_len; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 445 | mbedtls_mps_size_t frag_backup_offset; |
| 446 | mbedtls_mps_size_t frag_backup_len; |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 447 | |
| 448 | mbedtls_mps_size_t backup_len; |
| 449 | mbedtls_mps_size_t acc_len_needed; |
| 450 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 451 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 452 | "There has been an unsatisfied read with %u bytes overhead.", |
| 453 | (unsigned) pending); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 454 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 455 | if (acc == NULL) { |
| 456 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 457 | "No accumulator present"); |
Hanno Becker | 984fbde | 2021-01-28 09:02:18 +0000 | [diff] [blame] | 458 | MBEDTLS_MPS_TRACE_RETURN( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 459 | MBEDTLS_ERR_MPS_READER_NEED_ACCUMULATOR); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 460 | } |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 461 | acc_len = rd->acc_len; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 462 | |
| 463 | /* Check if the upper layer has already fetched |
| 464 | * and committed the contents of the accumulator. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 465 | if (commit < frag_offset) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 466 | /* No, accumulator is still being processed. */ |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 467 | frag_backup_offset = 0; |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 468 | frag_backup_len = frag_len; |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 469 | acc_backup_offset = commit; |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 470 | acc_backup_len = frag_offset - commit; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 471 | } else { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 472 | /* Yes, the accumulator is already processed. */ |
Hanno Becker | 1682a8b | 2021-02-22 16:38:56 +0000 | [diff] [blame] | 473 | frag_backup_offset = commit - frag_offset; |
| 474 | frag_backup_len = frag_len - frag_backup_offset; |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 475 | acc_backup_offset = 0; |
| 476 | acc_backup_len = 0; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 477 | } |
| 478 | |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 479 | backup_len = acc_backup_len + frag_backup_len; |
| 480 | acc_len_needed = backup_len + pending; |
| 481 | |
| 482 | overflow = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 483 | overflow |= (backup_len < acc_backup_len); |
| 484 | overflow |= (acc_len_needed < backup_len); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 485 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 486 | if (overflow || acc_len < acc_len_needed) { |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 487 | /* Except for the different return code, we behave as if |
| 488 | * there hadn't been a call to mbedtls_mps_reader_get() |
| 489 | * since the last commit. */ |
| 490 | rd->end = commit; |
| 491 | rd->pending = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 492 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_ERROR, |
| 493 | "The accumulator is too small to handle the backup."); |
| 494 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_ERROR, |
| 495 | "* Size: %u", (unsigned) acc_len); |
| 496 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_ERROR, |
| 497 | "* Needed: %u (%u + %u)", |
| 498 | (unsigned) acc_len_needed, |
| 499 | (unsigned) backup_len, (unsigned) pending); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 500 | MBEDTLS_MPS_TRACE_RETURN( |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 501 | MBEDTLS_ERR_MPS_READER_ACCUMULATOR_TOO_SMALL); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 502 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 503 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 504 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 505 | "Fragment backup: %u", (unsigned) frag_backup_len); |
| 506 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 507 | "Accumulator backup: %u", (unsigned) acc_backup_len); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 508 | |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 509 | /* Move uncommitted parts from the accumulator to the front |
| 510 | * of the accumulator. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 511 | memmove(acc, acc + acc_backup_offset, acc_backup_len); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 512 | |
Tom Cosgrove | 1797b05 | 2022-12-04 17:19:59 +0000 | [diff] [blame] | 513 | /* Copy uncommitted parts of the current fragment to the |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 514 | * accumulator. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 515 | memcpy(acc + acc_backup_len, |
| 516 | frag + frag_backup_offset, frag_backup_len); |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 517 | |
Hanno Becker | b185543 | 2021-02-08 08:07:35 +0000 | [diff] [blame] | 518 | rd->acc_available = backup_len; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 519 | rd->acc_share.acc_remaining = pending; |
| 520 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 521 | if (paused != NULL) { |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 522 | *paused = 1; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 523 | } |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 524 | } |
| 525 | |
| 526 | rd->frag = NULL; |
| 527 | rd->frag_len = 0; |
| 528 | |
Hanno Becker | 4f84e20 | 2021-02-08 06:54:30 +0000 | [diff] [blame] | 529 | rd->commit = 0; |
| 530 | rd->end = 0; |
| 531 | rd->pending = 0; |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 532 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 533 | MBEDTLS_MPS_TRACE(MBEDTLS_MPS_TRACE_TYPE_COMMENT, |
| 534 | "Final state: aa %u, al %u, ar %u", |
| 535 | (unsigned) rd->acc_available, (unsigned) rd->acc_len, |
| 536 | (unsigned) rd->acc_share.acc_remaining); |
| 537 | MBEDTLS_MPS_TRACE_RETURN(0); |
Hanno Becker | 13cd784 | 2021-01-12 07:08:33 +0000 | [diff] [blame] | 538 | } |
Hanno Becker | 43c8f8c | 2021-03-05 05:16:45 +0000 | [diff] [blame] | 539 | |
Ronald Cron | 6f135e1 | 2021-12-08 16:57:54 +0100 | [diff] [blame] | 540 | #endif /* MBEDTLS_SSL_PROTO_TLS1_3 */ |