Mock push_request_errors() appended the errors in reverse order

confluentinc · Oct 11, 2021 · 063a9ae · 063a9ae
1 parent 2d78e92
commit 063a9ae
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/rdkafka_mock.c b/src/rdkafka_mock.c
@@ -1608,6 +1608,7 @@ rd_kafka_mock_push_request_errors_array (rd_kafka_mock_cluster_t *mcluster,
                                          const rd_kafka_resp_err_t *errors) {
         rd_kafka_mock_error_stack_t *errstack;
         size_t totcnt;
+        size_t i;
 
         mtx_lock(&mcluster->lock);
 
@@ -1622,8 +1623,8 @@ rd_kafka_mock_push_request_errors_array (rd_kafka_mock_cluster_t *mcluster,
                                             sizeof(*errstack->errs));
         }
 
-        while (cnt > 0) {
-                errstack->errs[errstack->cnt].err = errors[--cnt];
+        for (i = 0 ; i < cnt ; i++) {
+                errstack->errs[errstack->cnt].err = errors[i];
                 errstack->errs[errstack->cnt++].rtt = 0;
         }