Fix: array and sequence field's 'elems' members can be left NULL

[babeltrace.git] / formats / ctf / types / sequence.c

diff --git a/formats/ctf/types/sequence.c b/formats/ctf/types/sequence.c
index 5dae7d0281d4610c586d00b2ee487e8ddb5a063d..df91a533f01a470e20f5ea24c176f3c793245e83 100644 (file)
--- a/formats/ctf/types/sequence.c
+++ b/formats/ctf/types/sequence.c
@@ -41,25 +41,17 @@ int ctf_sequence_read(struct bt_stream_pos *ppos, struct bt_definition *definiti
                struct declaration_integer *integer_declaration =
                        container_of(elem, struct declaration_integer, p);
 
-               if (integer_declaration->encoding == CTF_STRING_UTF8
-                     || integer_declaration->encoding == CTF_STRING_ASCII) {
+               if (bt_int_is_char(elem)) {
+                       uint64_t len = bt_sequence_len(sequence_definition);
 
-                       if (integer_declaration->len == CHAR_BIT
-                           && integer_declaration->p.alignment == CHAR_BIT) {
-                               uint64_t len = bt_sequence_len(sequence_definition);
+                       if (!ctf_align_pos(pos, integer_declaration->p.alignment))
+                               return -EFAULT;
+                       if (!ctf_pos_access_ok(pos, len * CHAR_BIT))
+                               return -EFAULT;
 
-                               if (!ctf_align_pos(pos, integer_declaration->p.alignment))
-                                       return -EFAULT;
-                               if (!ctf_pos_access_ok(pos, len * CHAR_BIT))
-                                       return -EFAULT;
-
-                               g_string_assign(sequence_definition->string, "");
-                               g_string_insert_len(sequence_definition->string,
-                                       0, (char *) ctf_get_pos_addr(pos), len);
-                               if (!ctf_move_pos(pos, len * CHAR_BIT))
-                                       return -EFAULT;
-                               return 0;
-                       }
+                       g_string_assign(sequence_definition->string, "");
+                       g_string_insert_len(sequence_definition->string,
+                               0, (char *) ctf_get_pos_addr(pos), len);
                }
        }
        return bt_sequence_rw(ppos, definition);
@@ -78,24 +70,19 @@ int ctf_sequence_write(struct bt_stream_pos *ppos, struct bt_definition *definit
                struct declaration_integer *integer_declaration =
                        container_of(elem, struct declaration_integer, p);
 
-               if (integer_declaration->encoding == CTF_STRING_UTF8
-                     || integer_declaration->encoding == CTF_STRING_ASCII) {
-
-                       if (integer_declaration->len == CHAR_BIT
-                           && integer_declaration->p.alignment == CHAR_BIT) {
-                               uint64_t len = bt_sequence_len(sequence_definition);
+               if (bt_int_is_char(elem)) {
+                       uint64_t len = bt_sequence_len(sequence_definition);
 
-                               if (!ctf_align_pos(pos, integer_declaration->p.alignment))
-                                       return -EFAULT;
-                               if (!ctf_pos_access_ok(pos, len * CHAR_BIT))
-                                       return -EFAULT;
+                       if (!ctf_align_pos(pos, integer_declaration->p.alignment))
+                               return -EFAULT;
+                       if (!ctf_pos_access_ok(pos, len * CHAR_BIT))
+                               return -EFAULT;
 
-                               memcpy((char *) ctf_get_pos_addr(pos),
-                                       sequence_definition->string->str, len);
-                               if (!ctf_move_pos(pos, len * CHAR_BIT))
-                                       return -EFAULT;
-                               return 0;
-                       }
+                       memcpy((char *) ctf_get_pos_addr(pos),
+                               sequence_definition->string->str, len);
+                       if (!ctf_move_pos(pos, len * CHAR_BIT))
+                               return -EFAULT;
+                       return 0;
                }
        }
        return bt_sequence_rw(ppos, definition);