Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 1 | /* BEGIN_HEADER */ |
| 2 | |
| 3 | #include <stdlib.h> |
| 4 | |
Hanno Becker | 8a04b10 | 2021-02-22 16:49:24 +0000 | [diff] [blame] | 5 | #include "mps_reader.h" |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 6 | |
| 7 | /* |
| 8 | * Compile-time configuration for test suite. |
| 9 | */ |
| 10 | |
| 11 | /* Comment/Uncomment this to disable/enable the |
| 12 | * testing of the various MPS layers. |
| 13 | * This can be useful for time-consuming instrumentation |
| 14 | * tasks such as the conversion of E-ACSL annotations |
| 15 | * into runtime assertions. */ |
| 16 | #define TEST_SUITE_MPS_READER |
| 17 | |
| 18 | /* End of compile-time configuration. */ |
| 19 | |
| 20 | /* END_HEADER */ |
| 21 | |
Hanno Becker | 43c8f8c | 2021-03-05 05:16:45 +0000 | [diff] [blame] | 22 | /* BEGIN_DEPENDENCIES |
Ronald Cron | 6f135e1 | 2021-12-08 16:57:54 +0100 | [diff] [blame] | 23 | * depends_on:MBEDTLS_SSL_PROTO_TLS1_3 |
Hanno Becker | 43c8f8c | 2021-03-05 05:16:45 +0000 | [diff] [blame] | 24 | * END_DEPENDENCIES |
| 25 | */ |
| 26 | |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 27 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 28 | void mbedtls_mps_reader_no_pausing_single_step_single_round(int with_acc) |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 29 | { |
| 30 | /* This test exercises the most basic use of the MPS reader: |
| 31 | * - The 'producing' layer provides a buffer |
| 32 | * - The 'consuming' layer fetches it in a single go. |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 33 | * - After processing, the consuming layer commits the data |
| 34 | * and the reader is moved back to producing mode. |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 35 | * |
| 36 | * Parameters: |
| 37 | * - with_acc: 0 if the reader should be initialized without accumulator. |
| 38 | * 1 if the reader should be initialized with accumulator. |
| 39 | * |
| 40 | * Whether the accumulator is present or not should not matter, |
| 41 | * since the consumer's request can be fulfilled from the data |
| 42 | * that the producer has provided. |
| 43 | */ |
| 44 | unsigned char bufA[100]; |
| 45 | unsigned char acc[10]; |
| 46 | unsigned char *tmp; |
Hanno Becker | d7fcbfa | 2021-03-08 16:25:38 +0000 | [diff] [blame] | 47 | int paused; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 48 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 49 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 50 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 51 | } |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 52 | |
| 53 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 54 | if (with_acc == 0) { |
| 55 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 56 | } else { |
| 57 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 58 | } |
| 59 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 60 | /* Consumption (upper layer) */ |
| 61 | /* Consume exactly what's available */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 62 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 63 | TEST_MEMORY_COMPARE(tmp, 100, bufA, 100); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 64 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 65 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 66 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, &paused) == 0); |
| 67 | TEST_ASSERT(paused == 0); |
| 68 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 09d880a | 2021-01-12 07:43:30 +0000 | [diff] [blame] | 69 | } |
| 70 | /* END_CASE */ |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 71 | |
| 72 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 73 | void mbedtls_mps_reader_no_pausing_single_step_multiple_rounds(int with_acc) |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 74 | { |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 75 | /* This test exercises multiple rounds of the basic use of the MPS reader: |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 76 | * - The 'producing' layer provides a buffer |
| 77 | * - The 'consuming' layer fetches it in a single go. |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 78 | * - After processing, the consuming layer commits the data |
| 79 | * and the reader is moved back to producing mode. |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 80 | * |
| 81 | * Parameters: |
| 82 | * - with_acc: 0 if the reader should be initialized without accumulator. |
| 83 | * 1 if the reader should be initialized with accumulator. |
| 84 | * |
| 85 | * Whether the accumulator is present or not should not matter, |
| 86 | * since the consumer's request can be fulfilled from the data |
| 87 | * that the producer has provided. |
| 88 | */ |
| 89 | |
| 90 | unsigned char bufA[100], bufB[100]; |
| 91 | unsigned char acc[10]; |
| 92 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 93 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 94 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 95 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 96 | } |
| 97 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 98 | bufB[i] = ~((unsigned char) i); |
| 99 | } |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 100 | |
| 101 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 102 | if (with_acc == 0) { |
| 103 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 104 | } else { |
| 105 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 106 | } |
| 107 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 108 | /* Consumption (upper layer) */ |
| 109 | /* Consume exactly what's available */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 110 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 111 | TEST_MEMORY_COMPARE(tmp, 100, bufA, 100); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 112 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 113 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 114 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 115 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, sizeof(bufB)) == 0); |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 116 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 117 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 118 | TEST_MEMORY_COMPARE(tmp, 100, bufB, 100); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 119 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 120 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 121 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 122 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 0e4edfc | 2021-01-12 07:52:29 +0000 | [diff] [blame] | 123 | } |
| 124 | /* END_CASE */ |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 125 | |
| 126 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 127 | void mbedtls_mps_reader_no_pausing_multiple_steps_single_round(int with_acc) |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 128 | { |
| 129 | /* This test exercises one round of the following: |
| 130 | * - The 'producing' layer provides a buffer |
| 131 | * - The 'consuming' layer fetches it in multiple calls |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 132 | * to `mbedtls_mps_reader_get()`, without committing in between. |
| 133 | * - After processing, the consuming layer commits the data |
| 134 | * and the reader is moved back to producing mode. |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 135 | * |
| 136 | * Parameters: |
| 137 | * - with_acc: 0 if the reader should be initialized without accumulator. |
| 138 | * 1 if the reader should be initialized with accumulator. |
| 139 | * |
| 140 | * Whether the accumulator is present or not should not matter, |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 141 | * since the consumer's requests can be fulfilled from the data |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 142 | * that the producer has provided. |
| 143 | */ |
| 144 | |
| 145 | /* Lower layer provides data that the upper layer fully consumes |
| 146 | * through multiple `get` calls. */ |
| 147 | unsigned char buf[100]; |
| 148 | unsigned char acc[10]; |
| 149 | unsigned char *tmp; |
| 150 | mbedtls_mps_size_t tmp_len; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 151 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 152 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 153 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 154 | } |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 155 | |
| 156 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 157 | if (with_acc == 0) { |
| 158 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 159 | } else { |
| 160 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 161 | } |
| 162 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 163 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 164 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 165 | TEST_MEMORY_COMPARE(tmp, 10, buf, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 166 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 70, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 167 | TEST_MEMORY_COMPARE(tmp, 70, buf + 10, 70); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 168 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 30, &tmp, &tmp_len) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 169 | TEST_MEMORY_COMPARE(tmp, tmp_len, buf + 80, 20); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 170 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 171 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 172 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 173 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | dbd8a96 | 2021-01-12 08:01:16 +0000 | [diff] [blame] | 174 | } |
| 175 | /* END_CASE */ |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 176 | |
| 177 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 178 | void mbedtls_mps_reader_no_pausing_multiple_steps_multiple_rounds(int with_acc) |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 179 | { |
| 180 | /* This test exercises one round of fetching a buffer in multiple chunks |
| 181 | * and passing it back to the producer afterwards, followed by another |
| 182 | * single-step sequence of feed-fetch-commit-reclaim. |
| 183 | */ |
| 184 | unsigned char bufA[100], bufB[100]; |
| 185 | unsigned char acc[10]; |
| 186 | unsigned char *tmp; |
| 187 | mbedtls_mps_size_t tmp_len; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 188 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 189 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 190 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 191 | } |
| 192 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 193 | bufB[i] = ~((unsigned char) i); |
| 194 | } |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 195 | |
| 196 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 197 | if (with_acc == 0) { |
| 198 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 199 | } else { |
| 200 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 201 | } |
| 202 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 203 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 204 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 205 | TEST_MEMORY_COMPARE(tmp, 10, bufA, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 206 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 70, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 207 | TEST_MEMORY_COMPARE(tmp, 70, bufA + 10, 70); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 208 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 30, &tmp, &tmp_len) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 209 | TEST_MEMORY_COMPARE(tmp, tmp_len, bufA + 80, 20); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 210 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 211 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 212 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 213 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, sizeof(bufB)) == 0); |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 214 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 215 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 216 | TEST_MEMORY_COMPARE(tmp, 100, bufB, 100); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 217 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 218 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 219 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 220 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 7973b2d | 2021-01-12 08:11:40 +0000 | [diff] [blame] | 221 | } |
| 222 | /* END_CASE */ |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 223 | |
| 224 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
| 225 | void mbedtls_mps_reader_pausing_needed_disabled() |
| 226 | { |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 227 | /* This test exercises the behaviour of the MPS reader when a read request |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 228 | * of the consumer exceeds what has been provided by the producer, and when |
| 229 | * no accumulator is available in the reader. |
| 230 | * |
| 231 | * In this case, we expect the reader to fail. |
| 232 | */ |
| 233 | |
| 234 | unsigned char buf[100]; |
| 235 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 236 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 237 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 238 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 239 | } |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 240 | |
| 241 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 242 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 243 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 244 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 245 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 246 | TEST_MEMORY_COMPARE(tmp, 50, buf, 50); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 247 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 248 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == |
| 249 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 250 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 251 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 252 | MBEDTLS_ERR_MPS_READER_NEED_ACCUMULATOR); |
| 253 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 7d86b74 | 2021-01-12 08:14:38 +0000 | [diff] [blame] | 254 | } |
| 255 | /* END_CASE */ |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 256 | |
| 257 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
| 258 | void mbedtls_mps_reader_pausing_needed_buffer_too_small() |
| 259 | { |
| 260 | /* This test exercises the behaviour of the MPS reader with accumulator |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 261 | * in the situation where a read request goes beyond the bounds of the |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 262 | * current read buffer, _and_ the reader's accumulator is too small to |
| 263 | * hold the requested amount of data. |
| 264 | * |
Hanno Becker | 014f683 | 2021-02-08 06:52:21 +0000 | [diff] [blame] | 265 | * In this case, we expect mbedtls_mps_reader_reclaim() to fail, |
| 266 | * but it should be possible to continue fetching data as if |
| 267 | * there had been no excess request via mbedtls_mps_reader_get() |
| 268 | * and the call to mbedtls_mps_reader_reclaim() had been rejected |
| 269 | * because of data remaining. |
| 270 | */ |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 271 | |
| 272 | unsigned char buf[100]; |
| 273 | unsigned char acc[10]; |
| 274 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 275 | mbedtls_mps_reader rd; |
Hanno Becker | 014f683 | 2021-02-08 06:52:21 +0000 | [diff] [blame] | 276 | mbedtls_mps_size_t tmp_len; |
| 277 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 278 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 279 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 280 | } |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 281 | |
| 282 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 283 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 284 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 285 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 286 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 287 | TEST_MEMORY_COMPARE(tmp, 50, buf, 50); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 288 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 289 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 290 | TEST_MEMORY_COMPARE(tmp, 10, buf + 50, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 291 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == |
| 292 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 293 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 294 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 295 | MBEDTLS_ERR_MPS_READER_ACCUMULATOR_TOO_SMALL); |
Hanno Becker | 014f683 | 2021-02-08 06:52:21 +0000 | [diff] [blame] | 296 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 297 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, &tmp_len) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 298 | TEST_MEMORY_COMPARE(tmp, tmp_len, buf + 50, 50); |
Hanno Becker | 014f683 | 2021-02-08 06:52:21 +0000 | [diff] [blame] | 299 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 300 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | caf1a3f | 2021-01-12 08:18:12 +0000 | [diff] [blame] | 301 | } |
| 302 | /* END_CASE */ |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 303 | |
| 304 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 305 | void mbedtls_mps_reader_reclaim_overflow() |
| 306 | { |
| 307 | /* This test exercises the behaviour of the MPS reader with accumulator |
| 308 | * in the situation where upon calling mbedtls_mps_reader_reclaim(), the |
| 309 | * uncommitted data together with the excess data missing in the last |
Tom Cosgrove | 1797b05 | 2022-12-04 17:19:59 +0000 | [diff] [blame] | 310 | * call to mbedtls_mps_reader_get() exceeds the bounds of the type |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 311 | * holding the buffer length. |
| 312 | */ |
| 313 | |
| 314 | unsigned char buf[100]; |
| 315 | unsigned char acc[50]; |
| 316 | unsigned char *tmp; |
| 317 | mbedtls_mps_reader rd; |
| 318 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 319 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | 5b3841d | 2021-03-19 05:23:30 +0000 | [diff] [blame] | 320 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 321 | } |
Hanno Becker | 5b3841d | 2021-03-19 05:23:30 +0000 | [diff] [blame] | 322 | |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 323 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 324 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 325 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 326 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 327 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 328 | TEST_MEMORY_COMPARE(tmp, 50, buf, 50); |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 329 | /* Excess request */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 330 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, (mbedtls_mps_size_t) -1, &tmp, NULL) == |
| 331 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 332 | /* Wrapup (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 333 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 334 | MBEDTLS_ERR_MPS_READER_ACCUMULATOR_TOO_SMALL); |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 335 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 336 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 1b1e7eb | 2021-03-08 16:57:08 +0000 | [diff] [blame] | 337 | } |
| 338 | /* END_CASE */ |
| 339 | |
| 340 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 341 | void mbedtls_mps_reader_pausing(int option) |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 342 | { |
| 343 | /* This test exercises the behaviour of the reader when the |
Hanno Becker | 756abeb | 2021-03-08 16:28:09 +0000 | [diff] [blame] | 344 | * accumulator is used to fulfill a consumer's request. |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 345 | * |
| 346 | * More detailed: |
| 347 | * - The producer feeds some data. |
| 348 | * - The consumer asks for more data than what's available. |
| 349 | * - The reader remembers the request and goes back to |
| 350 | * producing mode, waiting for more data from the producer. |
| 351 | * - The producer provides another chunk of data which is |
| 352 | * sufficient to fulfill the original read request. |
| 353 | * - The consumer retries the original read request, which |
| 354 | * should now succeed. |
| 355 | * |
| 356 | * This test comes in multiple variants controlled by the |
| 357 | * `option` parameter and documented below. |
| 358 | */ |
| 359 | |
| 360 | unsigned char bufA[100], bufB[100]; |
| 361 | unsigned char *tmp; |
| 362 | unsigned char acc[40]; |
Hanno Becker | d7fcbfa | 2021-03-08 16:25:38 +0000 | [diff] [blame] | 363 | int paused; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 364 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 365 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 366 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 367 | } |
| 368 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 369 | bufB[i] = ~((unsigned char) i); |
| 370 | } |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 371 | |
| 372 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 373 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 374 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 375 | |
| 376 | /* Consumption (upper layer) */ |
| 377 | /* Ask for more than what's available. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 378 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 80, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 379 | TEST_MEMORY_COMPARE(tmp, 80, bufA, 80); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 380 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 381 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 382 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 383 | switch (option) { |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 384 | case 0: /* Single uncommitted fetch at pausing */ |
| 385 | case 1: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 386 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 387 | break; |
| 388 | default: /* Multiple uncommitted fetches at pausing */ |
| 389 | break; |
| 390 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 391 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == |
| 392 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 393 | |
| 394 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 395 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, &paused) == 0); |
| 396 | TEST_ASSERT(paused == 1); |
| 397 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, sizeof(bufB)) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 398 | |
| 399 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 400 | switch (option) { |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 401 | case 0: /* Single fetch at pausing, re-fetch with commit. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 402 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 403 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 404 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 405 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 406 | break; |
| 407 | |
| 408 | case 1: /* Single fetch at pausing, re-fetch without commit. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 409 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 410 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 411 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 412 | break; |
| 413 | |
| 414 | case 2: /* Multiple fetches at pausing, repeat without commit. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 415 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 416 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 417 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 418 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 419 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 420 | break; |
| 421 | |
| 422 | case 3: /* Multiple fetches at pausing, repeat with commit 1. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 423 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 424 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 425 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 426 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 427 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 428 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 429 | break; |
| 430 | |
| 431 | case 4: /* Multiple fetches at pausing, repeat with commit 2. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 432 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 433 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 434 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 435 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 436 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 437 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 438 | break; |
| 439 | |
| 440 | case 5: /* Multiple fetches at pausing, repeat with commit 3. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 441 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 442 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 443 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 444 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 445 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 446 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 447 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 448 | break; |
| 449 | |
| 450 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 451 | TEST_ASSERT(0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 452 | } |
| 453 | |
| 454 | /* In all cases, fetch the rest of the second buffer. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 455 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 90, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 456 | TEST_MEMORY_COMPARE(tmp, 90, bufB + 10, 90); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 457 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 458 | |
| 459 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 460 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 461 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | e82952a | 2021-01-12 08:27:29 +0000 | [diff] [blame] | 462 | } |
| 463 | /* END_CASE */ |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 464 | |
| 465 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 466 | void mbedtls_mps_reader_pausing_multiple_feeds(int option) |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 467 | { |
| 468 | /* This test exercises the behaviour of the MPS reader |
| 469 | * in the following situation: |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 470 | * - The consumer has asked for more than what's available, so the |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 471 | * reader pauses and waits for further input data via |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 472 | * `mbedtls_mps_reader_feed()` |
| 473 | * - Multiple such calls to `mbedtls_mps_reader_feed()` are necessary |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 474 | * to fulfill the original request, and the reader needs to do |
| 475 | * the necessary bookkeeping under the hood. |
| 476 | * |
| 477 | * This test comes in a few variants differing in the number and |
| 478 | * size of feed calls that the producer issues while the reader is |
| 479 | * accumulating the necessary data - see the comments below. |
| 480 | */ |
| 481 | |
| 482 | unsigned char bufA[100], bufB[100]; |
| 483 | unsigned char *tmp; |
| 484 | unsigned char acc[70]; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 485 | mbedtls_mps_reader rd; |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 486 | mbedtls_mps_size_t fetch_len; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 487 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 488 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 489 | } |
| 490 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 491 | bufB[i] = ~((unsigned char) i); |
| 492 | } |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 493 | |
| 494 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 495 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 496 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 497 | |
| 498 | /* Consumption (upper layer) */ |
| 499 | /* Ask for more than what's available. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 500 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 80, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 501 | TEST_MEMORY_COMPARE(tmp, 80, bufA, 80); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 502 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 503 | /* 20 left, ask for 70 -> 50 overhead */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 504 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 70, &tmp, NULL) == |
| 505 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 506 | |
| 507 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 508 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 509 | switch (option) { |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 510 | case 0: /* 10 + 10 + 80 byte feed */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 511 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, 10) == |
| 512 | MBEDTLS_ERR_MPS_READER_NEED_MORE); |
| 513 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + 10, 10) == |
| 514 | MBEDTLS_ERR_MPS_READER_NEED_MORE); |
| 515 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + 20, 80) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 516 | break; |
| 517 | |
| 518 | case 1: /* 50 x 1byte */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 519 | for (size_t num_feed = 0; num_feed < 49; num_feed++) { |
| 520 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + num_feed, 1) == |
| 521 | MBEDTLS_ERR_MPS_READER_NEED_MORE); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 522 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 523 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + 49, 1) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 524 | break; |
| 525 | |
| 526 | case 2: /* 49 x 1byte + 51bytes */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 527 | for (size_t num_feed = 0; num_feed < 49; num_feed++) { |
| 528 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + num_feed, 1) == |
| 529 | MBEDTLS_ERR_MPS_READER_NEED_MORE); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 530 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 531 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB + 49, 51) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 532 | break; |
| 533 | |
| 534 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 535 | TEST_ASSERT(0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 536 | break; |
| 537 | } |
| 538 | |
| 539 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 540 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 70, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 541 | TEST_MEMORY_COMPARE(tmp, 20, bufA + 80, 20); |
| 542 | TEST_MEMORY_COMPARE(tmp + 20, 50, bufB, 50); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 543 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 1000, &tmp, &fetch_len) == 0); |
| 544 | switch (option) { |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 545 | case 0: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 546 | TEST_ASSERT(fetch_len == 50); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 547 | break; |
| 548 | |
| 549 | case 1: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 550 | TEST_ASSERT(fetch_len == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 551 | break; |
| 552 | |
| 553 | case 2: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 554 | TEST_ASSERT(fetch_len == 50); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 555 | break; |
| 556 | |
| 557 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 558 | TEST_ASSERT(0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 559 | break; |
| 560 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 561 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 562 | |
| 563 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 564 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 565 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | aac4122 | 2021-01-12 08:36:36 +0000 | [diff] [blame] | 566 | } |
| 567 | /* END_CASE */ |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 568 | |
| 569 | |
| 570 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 571 | void mbedtls_mps_reader_reclaim_data_left(int option) |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 572 | { |
| 573 | /* This test exercises the behaviour of the MPS reader when a |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 574 | * call to mbedtls_mps_reader_reclaim() is made before all data |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 575 | * provided by the producer has been fetched and committed. */ |
| 576 | |
| 577 | unsigned char buf[100]; |
| 578 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 579 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 580 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 581 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 582 | } |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 583 | |
| 584 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 585 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 586 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 587 | |
| 588 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 589 | switch (option) { |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 590 | case 0: |
| 591 | /* Fetch (but not commit) the entire buffer. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 592 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, sizeof(buf), &tmp, NULL) |
| 593 | == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 594 | TEST_MEMORY_COMPARE(tmp, 100, buf, 100); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 595 | break; |
| 596 | |
| 597 | case 1: |
| 598 | /* Fetch (but not commit) parts of the buffer. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 599 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, sizeof(buf) / 2, |
| 600 | &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 601 | TEST_MEMORY_COMPARE(tmp, sizeof(buf) / 2, buf, sizeof(buf) / 2); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 602 | break; |
| 603 | |
| 604 | case 2: |
| 605 | /* Fetch and commit parts of the buffer, then |
| 606 | * fetch but not commit the rest of the buffer. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 607 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, sizeof(buf) / 2, |
| 608 | &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 609 | TEST_MEMORY_COMPARE(tmp, sizeof(buf) / 2, buf, sizeof(buf) / 2); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 610 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 611 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, sizeof(buf) / 2, |
| 612 | &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 613 | TEST_MEMORY_COMPARE(tmp, sizeof(buf) / 2, |
Tom Cosgrove | 0540fe7 | 2023-07-27 14:17:27 +0100 | [diff] [blame^] | 614 | buf + sizeof(buf) / 2, |
| 615 | sizeof(buf) / 2); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 616 | break; |
| 617 | |
| 618 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 619 | TEST_ASSERT(0); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 620 | break; |
| 621 | } |
| 622 | |
| 623 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 624 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 625 | MBEDTLS_ERR_MPS_READER_DATA_LEFT); |
| 626 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | cb2a88e | 2021-01-12 08:39:37 +0000 | [diff] [blame] | 627 | } |
| 628 | /* END_CASE */ |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 629 | |
| 630 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
| 631 | void mbedtls_mps_reader_reclaim_data_left_retry() |
| 632 | { |
| 633 | /* This test exercises the behaviour of the MPS reader when an attempt |
| 634 | * by the producer to reclaim the reader fails because of more data pending |
| 635 | * to be processed, and the consumer subsequently fetches more data. */ |
| 636 | unsigned char buf[100]; |
| 637 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 638 | mbedtls_mps_reader rd; |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 639 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 640 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 641 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 642 | } |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 643 | |
| 644 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 645 | mbedtls_mps_reader_init(&rd, NULL, 0); |
| 646 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 647 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 648 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 649 | TEST_MEMORY_COMPARE(tmp, 50, buf, 50); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 650 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 651 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 652 | TEST_MEMORY_COMPARE(tmp, 50, buf + 50, 50); |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 653 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 654 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 655 | MBEDTLS_ERR_MPS_READER_DATA_LEFT); |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 656 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 657 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 658 | TEST_MEMORY_COMPARE(tmp, 50, buf + 50, 50); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 659 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 660 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 661 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 662 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | e1f173c | 2021-01-12 08:43:58 +0000 | [diff] [blame] | 663 | } |
| 664 | /* END_CASE */ |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 665 | |
| 666 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 667 | void mbedtls_mps_reader_multiple_pausing(int option) |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 668 | { |
| 669 | /* This test exercises the behaviour of the MPS reader |
| 670 | * in the following situation: |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 671 | * - A read request via `mbedtls_mps_reader_get()` can't |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 672 | * be served and the reader is paused to accumulate |
| 673 | * the desired amount of data from the producer. |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 674 | * - Once enough data is available, the consumer successfully |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 675 | * reads the data from the reader, but afterwards exceeds |
| 676 | * the available data again - pausing is necessary for a |
| 677 | * second time. |
| 678 | */ |
| 679 | |
| 680 | unsigned char bufA[100], bufB[20], bufC[10]; |
| 681 | unsigned char *tmp; |
| 682 | unsigned char acc[50]; |
| 683 | mbedtls_mps_size_t tmp_len; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 684 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 685 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 686 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 687 | } |
| 688 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 689 | bufB[i] = ~((unsigned char) i); |
| 690 | } |
| 691 | for (size_t i = 0; (unsigned) i < sizeof(bufC); i++) { |
| 692 | bufC[i] = ~((unsigned char) i); |
| 693 | } |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 694 | |
| 695 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 696 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 697 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 698 | |
| 699 | /* Consumption (upper layer) */ |
| 700 | /* Ask for more than what's available. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 701 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 80, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 702 | TEST_MEMORY_COMPARE(tmp, 80, bufA, 80); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 703 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 704 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 705 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 706 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == |
| 707 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 708 | |
| 709 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 710 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 711 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, sizeof(bufB)) == 0); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 712 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 713 | switch (option) { |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 714 | case 0: /* Fetch same chunks, commit afterwards, and |
| 715 | * then exceed bounds of new buffer; accumulator |
| 716 | * large enough. */ |
| 717 | |
| 718 | /* Consume */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 719 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, &tmp_len) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 720 | TEST_MEMORY_COMPARE(tmp, tmp_len, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 721 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 722 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 723 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 724 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 725 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == |
| 726 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 727 | |
| 728 | /* Prepare */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 729 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 730 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufC, sizeof(bufC)) == 0);; |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 731 | |
| 732 | /* Consume */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 733 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 734 | TEST_MEMORY_COMPARE(tmp, 10, bufB + 10, 10); |
| 735 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufC, 10); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 736 | break; |
| 737 | |
| 738 | case 1: /* Fetch same chunks, commit afterwards, and |
| 739 | * then exceed bounds of new buffer; accumulator |
| 740 | * not large enough. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 741 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 742 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 743 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 744 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 745 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 746 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 747 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 51, &tmp, NULL) == |
| 748 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 749 | |
| 750 | /* Prepare */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 751 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 752 | MBEDTLS_ERR_MPS_READER_ACCUMULATOR_TOO_SMALL); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 753 | break; |
| 754 | |
| 755 | case 2: /* Fetch same chunks, don't commit afterwards, and |
| 756 | * then exceed bounds of new buffer; accumulator |
| 757 | * large enough. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 758 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 759 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 760 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 761 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 762 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 763 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == |
| 764 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 765 | |
| 766 | /* Prepare */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 767 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 768 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufC, sizeof(bufC)) == 0);; |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 769 | |
| 770 | /* Consume */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 771 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 50, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 772 | TEST_MEMORY_COMPARE(tmp, 20, bufA + 80, 20); |
| 773 | TEST_MEMORY_COMPARE(tmp + 20, 20, bufB, 20); |
| 774 | TEST_MEMORY_COMPARE(tmp + 40, 10, bufC, 10); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 775 | break; |
| 776 | |
| 777 | case 3: /* Fetch same chunks, don't commit afterwards, and |
| 778 | * then exceed bounds of new buffer; accumulator |
| 779 | * not large enough. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 780 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 781 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 80, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 782 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 783 | TEST_MEMORY_COMPARE(tmp, 10, bufA + 90, 10); |
| 784 | TEST_MEMORY_COMPARE(tmp + 10, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 785 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 21, &tmp, NULL) == |
| 786 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 787 | |
| 788 | /* Prepare */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 789 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == |
| 790 | MBEDTLS_ERR_MPS_READER_ACCUMULATOR_TOO_SMALL); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 791 | break; |
| 792 | |
| 793 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 794 | TEST_ASSERT(0); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 795 | break; |
| 796 | } |
| 797 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 798 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | b6fdd35 | 2021-01-12 09:17:56 +0000 | [diff] [blame] | 799 | } |
| 800 | /* END_CASE */ |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 801 | |
| 802 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER:MBEDTLS_MPS_STATE_VALIDATION */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 803 | void mbedtls_mps_reader_random_usage(int num_out_chunks, |
| 804 | int max_chunk_size, |
| 805 | int max_request, |
| 806 | int acc_size) |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 807 | |
| 808 | { |
| 809 | /* Randomly pass a reader object back and forth between lower and |
| 810 | * upper layer and let each of them call the respective reader API |
| 811 | * functions in a random fashion. |
| 812 | * |
| 813 | * On the lower layer, we're tracking and concatenating |
| 814 | * the data passed to successful feed calls. |
| 815 | * |
| 816 | * For the upper layer, we track and concatenate buffers |
| 817 | * obtained from successful get calls. |
| 818 | * |
| 819 | * As long as the lower layer calls reclaim at least once, (resetting the |
| 820 | * fetched but not-yet-committed data), this should always lead to the same |
| 821 | * stream of outgoing/incoming data for the lower/upper layers, even if |
| 822 | * most of the random calls fail. |
| 823 | * |
| 824 | * NOTE: This test uses rand() for random data, which is not optimal. |
| 825 | * Instead, it would be better to get the random data from a |
| 826 | * static buffer. This both eases reproducibility and allows |
| 827 | * simple conversion to a fuzz target. |
| 828 | */ |
| 829 | int ret; |
| 830 | unsigned char *acc = NULL; |
| 831 | unsigned char *outgoing = NULL, *incoming = NULL; |
| 832 | unsigned char *cur_chunk = NULL; |
| 833 | size_t cur_out_chunk, out_pos, in_commit, in_fetch; |
| 834 | int rand_op; /* Lower layer: |
| 835 | * - Reclaim (0) |
| 836 | * - Feed (1) |
| 837 | * Upper layer: |
| 838 | * - Get, do tolerate smaller output (0) |
| 839 | * - Get, don't tolerate smaller output (1) |
| 840 | * - Commit (2) */ |
| 841 | int mode = 0; /* Lower layer (0) or Upper layer (1) */ |
| 842 | int reclaimed = 1; /* Have to call reclaim at least once before |
| 843 | * returning the reader to the upper layer. */ |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 844 | mbedtls_mps_reader rd; |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 845 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 846 | if (acc_size > 0) { |
Tom Cosgrove | 05b2a87 | 2023-07-21 11:31:13 +0100 | [diff] [blame] | 847 | TEST_CALLOC(acc, acc_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 848 | } |
| 849 | |
| 850 | /* This probably needs to be changed because we want |
| 851 | * our tests to be deterministic. */ |
| 852 | // srand( time( NULL ) ); |
| 853 | |
Tom Cosgrove | 05b2a87 | 2023-07-21 11:31:13 +0100 | [diff] [blame] | 854 | TEST_CALLOC(outgoing, num_out_chunks * max_chunk_size); |
| 855 | TEST_CALLOC(incoming, num_out_chunks * max_chunk_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 856 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 857 | mbedtls_mps_reader_init(&rd, acc, acc_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 858 | |
| 859 | cur_out_chunk = 0; |
| 860 | in_commit = 0; |
| 861 | in_fetch = 0; |
| 862 | out_pos = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 863 | while (cur_out_chunk < (unsigned) num_out_chunks) { |
| 864 | if (mode == 0) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 865 | /* Choose randomly between reclaim and feed */ |
| 866 | rand_op = rand() % 2; |
| 867 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 868 | if (rand_op == 0) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 869 | /* Reclaim */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 870 | ret = mbedtls_mps_reader_reclaim(&rd, NULL); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 871 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 872 | if (ret == 0) { |
| 873 | TEST_ASSERT(cur_chunk != NULL); |
| 874 | mbedtls_free(cur_chunk); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 875 | cur_chunk = NULL; |
| 876 | } |
| 877 | reclaimed = 1; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 878 | } else { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 879 | /* Feed reader with a random chunk */ |
| 880 | unsigned char *tmp = NULL; |
| 881 | size_t tmp_size; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 882 | if (cur_out_chunk == (unsigned) num_out_chunks) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 883 | continue; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 884 | } |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 885 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 886 | tmp_size = (rand() % max_chunk_size) + 1; |
Tom Cosgrove | 05b2a87 | 2023-07-21 11:31:13 +0100 | [diff] [blame] | 887 | TEST_CALLOC(tmp, tmp_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 888 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 889 | TEST_ASSERT(mbedtls_test_rnd_std_rand(NULL, tmp, tmp_size) == 0); |
| 890 | ret = mbedtls_mps_reader_feed(&rd, tmp, tmp_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 891 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 892 | if (ret == 0 || ret == MBEDTLS_ERR_MPS_READER_NEED_MORE) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 893 | cur_out_chunk++; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 894 | memcpy(outgoing + out_pos, tmp, tmp_size); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 895 | out_pos += tmp_size; |
| 896 | } |
| 897 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 898 | if (ret == 0) { |
| 899 | TEST_ASSERT(cur_chunk == NULL); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 900 | cur_chunk = tmp; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 901 | } else { |
| 902 | mbedtls_free(tmp); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 903 | } |
| 904 | |
| 905 | } |
| 906 | |
| 907 | /* Randomly switch to consumption mode if reclaim |
| 908 | * was called at least once. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 909 | if (reclaimed == 1 && rand() % 3 == 0) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 910 | in_fetch = 0; |
| 911 | mode = 1; |
| 912 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 913 | } else { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 914 | /* Choose randomly between get tolerating fewer data, |
| 915 | * get not tolerating fewer data, and commit. */ |
| 916 | rand_op = rand() % 3; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 917 | if (rand_op == 0 || rand_op == 1) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 918 | mbedtls_mps_size_t get_size, real_size; |
| 919 | unsigned char *chunk_get; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 920 | get_size = (rand() % max_request) + 1; |
| 921 | if (rand_op == 0) { |
| 922 | ret = mbedtls_mps_reader_get(&rd, get_size, &chunk_get, |
| 923 | &real_size); |
| 924 | } else { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 925 | real_size = get_size; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 926 | ret = mbedtls_mps_reader_get(&rd, get_size, &chunk_get, NULL); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 927 | } |
| 928 | |
| 929 | /* Check if output is in accordance with what was written */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 930 | if (ret == 0) { |
| 931 | memcpy(incoming + in_commit + in_fetch, |
| 932 | chunk_get, real_size); |
| 933 | TEST_ASSERT(memcmp(incoming + in_commit + in_fetch, |
| 934 | outgoing + in_commit + in_fetch, |
| 935 | real_size) == 0); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 936 | in_fetch += real_size; |
| 937 | } |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 938 | } else if (rand_op == 2) { /* Commit */ |
| 939 | ret = mbedtls_mps_reader_commit(&rd); |
| 940 | if (ret == 0) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 941 | in_commit += in_fetch; |
| 942 | in_fetch = 0; |
| 943 | } |
| 944 | } |
| 945 | |
| 946 | /* Randomly switch back to preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 947 | if (rand() % 3 == 0) { |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 948 | reclaimed = 0; |
| 949 | mode = 0; |
| 950 | } |
| 951 | } |
| 952 | } |
| 953 | |
| 954 | /* Cleanup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 955 | mbedtls_mps_reader_free(&rd); |
| 956 | mbedtls_free(incoming); |
| 957 | mbedtls_free(outgoing); |
| 958 | mbedtls_free(acc); |
| 959 | mbedtls_free(cur_chunk); |
Hanno Becker | 714cbeb | 2021-01-12 09:23:15 +0000 | [diff] [blame] | 960 | } |
| 961 | /* END_CASE */ |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 962 | |
| 963 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 964 | void mbedtls_reader_inconsistent_usage(int option) |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 965 | { |
| 966 | /* This test exercises the behaviour of the MPS reader |
| 967 | * in the following situation: |
| 968 | * - The consumer asks for more data than what's available |
| 969 | * - The reader is paused and receives more data from the |
| 970 | * producer until the original read request can be fulfilled. |
| 971 | * - The consumer does not repeat the original request but |
| 972 | * requests data in a different way. |
| 973 | * |
| 974 | * The reader does not guarantee that inconsistent read requests |
| 975 | * after pausing will succeed, and this test triggers some cases |
| 976 | * where the request fails. |
| 977 | */ |
| 978 | |
| 979 | unsigned char bufA[100], bufB[100]; |
| 980 | unsigned char *tmp; |
| 981 | unsigned char acc[40]; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 982 | mbedtls_mps_reader rd; |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 983 | int success = 0; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 984 | for (size_t i = 0; (unsigned) i < sizeof(bufA); i++) { |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 985 | bufA[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 986 | } |
| 987 | for (size_t i = 0; (unsigned) i < sizeof(bufB); i++) { |
| 988 | bufB[i] = ~((unsigned char) i); |
| 989 | } |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 990 | |
| 991 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 992 | mbedtls_mps_reader_init(&rd, acc, sizeof(acc)); |
| 993 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufA, sizeof(bufA)) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 994 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 995 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 80, &tmp, NULL) == 0); |
| 996 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 997 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == 0); |
| 998 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 20, &tmp, NULL) == |
| 999 | MBEDTLS_ERR_MPS_READER_OUT_OF_DATA); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1000 | /* Preparation */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1001 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 1002 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, bufB, sizeof(bufB)) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1003 | /* Consumption */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1004 | switch (option) { |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1005 | case 0: |
| 1006 | /* Ask for buffered data in a single chunk, no commit */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1007 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 30, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1008 | TEST_MEMORY_COMPARE(tmp, 20, bufA + 80, 20); |
| 1009 | TEST_MEMORY_COMPARE(tmp + 20, 10, bufB, 10); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1010 | success = 1; |
| 1011 | break; |
| 1012 | |
| 1013 | case 1: |
| 1014 | /* Ask for buffered data in a single chunk, with commit */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1015 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 30, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1016 | TEST_MEMORY_COMPARE(tmp, 20, bufA + 80, 20); |
| 1017 | TEST_MEMORY_COMPARE(tmp + 20, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1018 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1019 | success = 1; |
| 1020 | break; |
| 1021 | |
| 1022 | case 2: |
| 1023 | /* Ask for more than was requested when pausing, #1 */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1024 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 31, &tmp, NULL) == |
| 1025 | MBEDTLS_ERR_MPS_READER_INCONSISTENT_REQUESTS); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1026 | break; |
| 1027 | |
| 1028 | case 3: |
| 1029 | /* Ask for more than was requested when pausing #2 */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1030 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, (mbedtls_mps_size_t) -1, &tmp, NULL) == |
| 1031 | MBEDTLS_ERR_MPS_READER_INCONSISTENT_REQUESTS); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1032 | break; |
| 1033 | |
| 1034 | case 4: |
| 1035 | /* Asking for buffered data in different |
| 1036 | * chunks than before CAN fail. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1037 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1038 | TEST_MEMORY_COMPARE(tmp, 15, bufA + 80, 15); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1039 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 10, &tmp, NULL) == |
| 1040 | MBEDTLS_ERR_MPS_READER_INCONSISTENT_REQUESTS); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1041 | break; |
| 1042 | |
| 1043 | case 5: |
| 1044 | /* Asking for buffered data different chunks |
| 1045 | * than before NEED NOT fail - no commits */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1046 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1047 | TEST_MEMORY_COMPARE(tmp, 15, bufA + 80, 15); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1048 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1049 | TEST_MEMORY_COMPARE(tmp, 5, bufA + 95, 5); |
| 1050 | TEST_MEMORY_COMPARE(tmp + 5, 10, bufB, 10); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1051 | success = 1; |
| 1052 | break; |
| 1053 | |
| 1054 | case 6: |
| 1055 | /* Asking for buffered data different chunks |
| 1056 | * than before NEED NOT fail - intermediate commit */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1057 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1058 | TEST_MEMORY_COMPARE(tmp, 15, bufA + 80, 15); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1059 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
| 1060 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1061 | TEST_MEMORY_COMPARE(tmp, 5, bufA + 95, 5); |
| 1062 | TEST_MEMORY_COMPARE(tmp + 5, 10, bufB, 10); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1063 | success = 1; |
| 1064 | break; |
| 1065 | |
| 1066 | case 7: |
| 1067 | /* Asking for buffered data different chunks |
| 1068 | * than before NEED NOT fail - end commit */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1069 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1070 | TEST_MEMORY_COMPARE(tmp, 15, bufA + 80, 15); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1071 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1072 | TEST_MEMORY_COMPARE(tmp, 5, bufA + 95, 5); |
| 1073 | TEST_MEMORY_COMPARE(tmp + 5, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1074 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1075 | success = 1; |
| 1076 | break; |
| 1077 | |
| 1078 | case 8: |
| 1079 | /* Asking for buffered data different chunks |
| 1080 | * than before NEED NOT fail - intermediate & end commit */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1081 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1082 | TEST_MEMORY_COMPARE(tmp, 15, bufA + 80, 15); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1083 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 15, &tmp, NULL) == 0); |
| 1084 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1085 | TEST_MEMORY_COMPARE(tmp, 5, bufA + 95, 5); |
| 1086 | TEST_MEMORY_COMPARE(tmp + 5, 10, bufB, 10); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1087 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1088 | success = 1; |
| 1089 | break; |
| 1090 | |
| 1091 | default: |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1092 | TEST_ASSERT(0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1093 | break; |
| 1094 | } |
| 1095 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1096 | if (success == 1) { |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1097 | /* In all succeeding cases, fetch the rest of the second buffer. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1098 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 90, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1099 | TEST_MEMORY_COMPARE(tmp, 90, bufB + 10, 90); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1100 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1101 | |
| 1102 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1103 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1104 | } |
| 1105 | |
| 1106 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1107 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 223b72e | 2021-01-12 09:31:31 +0000 | [diff] [blame] | 1108 | } |
| 1109 | /* END_CASE */ |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1110 | |
| 1111 | /* BEGIN_CASE depends_on:TEST_SUITE_MPS_READER */ |
Hanno Becker | 15da2fc | 2021-02-22 16:57:14 +0000 | [diff] [blame] | 1112 | void mbedtls_mps_reader_feed_empty() |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1113 | { |
| 1114 | /* This test exercises the behaviour of the reader when it is |
Hanno Becker | 5047b56 | 2021-02-22 16:52:02 +0000 | [diff] [blame] | 1115 | * fed with a NULL buffer. */ |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1116 | unsigned char buf[100]; |
| 1117 | unsigned char *tmp; |
Hanno Becker | 8899396 | 2021-01-28 09:45:47 +0000 | [diff] [blame] | 1118 | mbedtls_mps_reader rd; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1119 | for (size_t i = 0; (unsigned) i < sizeof(buf); i++) { |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1120 | buf[i] = (unsigned char) i; |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1121 | } |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1122 | |
| 1123 | /* Preparation (lower layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1124 | mbedtls_mps_reader_init(&rd, NULL, 0); |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1125 | |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1126 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, NULL, sizeof(buf)) == |
| 1127 | MBEDTLS_ERR_MPS_READER_INVALID_ARG); |
Hanno Becker | 15da2fc | 2021-02-22 16:57:14 +0000 | [diff] [blame] | 1128 | |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1129 | /* Subsequent feed-calls should still succeed. */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1130 | TEST_ASSERT(mbedtls_mps_reader_feed(&rd, buf, sizeof(buf)) == 0); |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1131 | |
| 1132 | /* Consumption (upper layer) */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1133 | TEST_ASSERT(mbedtls_mps_reader_get(&rd, 100, &tmp, NULL) == 0); |
Tom Cosgrove | e4e9e7d | 2023-07-21 11:40:20 +0100 | [diff] [blame] | 1134 | TEST_MEMORY_COMPARE(tmp, 100, buf, 100); |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1135 | TEST_ASSERT(mbedtls_mps_reader_commit(&rd) == 0); |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1136 | |
| 1137 | /* Wrapup */ |
Gilles Peskine | 449bd83 | 2023-01-11 14:50:10 +0100 | [diff] [blame] | 1138 | TEST_ASSERT(mbedtls_mps_reader_reclaim(&rd, NULL) == 0); |
| 1139 | mbedtls_mps_reader_free(&rd); |
Hanno Becker | 2b8bad3 | 2021-01-12 09:40:05 +0000 | [diff] [blame] | 1140 | } |
| 1141 | /* END_CASE */ |