opentelemetry: logs: strict check when decoding IDs

Signed-off-by: Eduardo Silva <[email protected]>

Author: edsiper

src/opentelemetry/flb_opentelemetry_logs.c

@@ -302,14 +302,24 @@ static int process_json_payload_log_records_entry(
     }
 
     if (trace_id != NULL && trace_id->type == MSGPACK_OBJECT_STR && trace_id->via.str.size == 32) {
-        flb_otel_utils_hex_to_id(trace_id->via.str.ptr, trace_id->via.str.size, tmp_id, 16);
+        if (flb_otel_utils_hex_to_id(trace_id->via.str.ptr, trace_id->via.str.size, tmp_id, 16) != 0) {
+            if (error_status) {
+                *error_status = FLB_OTEL_LOGS_ERR_INVALID_TRACE_ID;
+            }
+            return -FLB_OTEL_LOGS_ERR_INVALID_TRACE_ID;
+        }
         flb_log_event_encoder_append_metadata_values(encoder,
                                                         FLB_LOG_EVENT_STRING_VALUE("trace_id", 8),
                                                         FLB_LOG_EVENT_BINARY_VALUE(tmp_id, 16));
     }
 
     if (span_id != NULL && span_id->type == MSGPACK_OBJECT_STR && span_id->via.str.size == 16) {
-        flb_otel_utils_hex_to_id(span_id->via.str.ptr, span_id->via.str.size, tmp_id, 8);
+        if (flb_otel_utils_hex_to_id(span_id->via.str.ptr, span_id->via.str.size, tmp_id, 8) != 0) {
+            if (error_status) {
+                *error_status = FLB_OTEL_LOGS_ERR_INVALID_SPAN_ID;
+            }
+            return -FLB_OTEL_LOGS_ERR_INVALID_SPAN_ID;
+        }
         flb_log_event_encoder_append_metadata_values(encoder,
                                                         FLB_LOG_EVENT_STRING_VALUE("span_id", 7),
                                                         FLB_LOG_EVENT_BINARY_VALUE(tmp_id, 8));