commit c320de4b46f8158c580ae257e57ee3569dfe5e90
author Andrew Stitcher <astitcher@apache.org> Thu Aug 08 16:17:52 2024 -0400
committer Andrew Stitcher <astitcher@apache.org> Thu Aug 08 17:04:35 2024 -0400
tree 9da8f6f33ff55f2d8c90f169354dd160a5106bcf
parent 7c08f80ed69745a4504b0c3e6325a4ae6baf2b3c

PROTON-2842: Correct value dumps for arbitrary described types

c/src/core/value_dump.c

diff --git a/c/src/core/value_dump.c b/c/src/core/value_dump.c
index 63e2066..33f5df9 100644
--- a/c/src/core/value_dump.c
+++ b/c/src/core/value_dump.c
@@ -346,6 +346,14 @@
 
 void pn_value_dump_described_list(uint32_t count, pn_bytes_t value, uint64_t dcode, pn_fixed_string_t *output) {
   uint32_t elements = 0;
+  uint8_t field_count;
+  uint8_t first_field_index;
+  bool known_descriptor = dcode>=FIELD_MIN && dcode<=FIELD_MAX;
+  if (known_descriptor) {
+    const pn_fields_t *fields = &FIELDS[dcode-FIELD_MIN];
+    field_count = fields->field_count;
+    first_field_index = fields->first_field_index;
+  }
   bool output_element = false;
   pn_fixed_string_addf(output, "[");
   while (value.size) {
@@ -356,10 +364,11 @@
       if (output_element) {
         pn_fixed_string_addf(output, ", ");
       }
-      const pn_fields_t *fields = &FIELDS[dcode-FIELD_MIN];
-      if (elements < fields->field_count) {
-        pn_fixed_string_addf(output, "%s=",
-                      (const char*)FIELD_STRINGPOOL.STRING0+FIELD_FIELDS[fields->first_field_index+elements]);
+      if (known_descriptor) {
+        if (elements < field_count) {
+          pn_fixed_string_addf(output, "%s=",
+                        (const char*)FIELD_STRINGPOOL.STRING0+FIELD_FIELDS[first_field_index+elements]);
+        }
       }
       size_t size = pni_value_dump(value, output);
       value = pn_bytes_advance(value, size);