lib: do not allow port to be removed when message iterators are active

[babeltrace.git] / lib / graph / port.c

diff --git a/lib/graph/port.c b/lib/graph/port.c
index 01ee3944fc62bc83d101545da44266055e966399..86d01b92ac82e0bc1ee6023772e375bc8629468c 100644 (file)
--- a/lib/graph/port.c
+++ b/lib/graph/port.c
@@ -24,6 +24,8 @@
 #define BT_LOG_TAG "PORT"
 #include <babeltrace/lib-logging-internal.h>
 
+#include <babeltrace/assert-internal.h>
+#include <babeltrace/assert-pre-internal.h>
 #include <babeltrace/graph/port-const.h>
 #include <babeltrace/graph/port-input-const.h>
 #include <babeltrace/graph/port-output-const.h>
@@ -34,10 +36,7 @@
 #include <babeltrace/graph/port-internal.h>
 #include <babeltrace/graph/connection-internal.h>
 #include <babeltrace/object-internal.h>
-#include <babeltrace/object.h>
 #include <babeltrace/compiler-internal.h>
-#include <babeltrace/assert-internal.h>
-#include <babeltrace/assert-pre-internal.h>
 
 static
 void destroy_port(struct bt_object *obj)
@@ -138,6 +137,36 @@ void bt_port_set_connection(struct bt_port *port,
                connection);
 }
 
+static inline
+bool port_connection_iterators_are_finalized(struct bt_port *port)
+{
+       bool ret = true;
+       struct bt_connection *conn = port->connection;
+       uint64_t i;
+
+       if (!conn) {
+               goto end;
+       }
+
+       for (i = 0; i < conn->iterators->len; i++) {
+               struct bt_self_component_port_input_message_iterator *iterator =
+                       conn->iterators->pdata[i];
+
+               BT_ASSERT(iterator);
+
+               if (iterator->state != BT_SELF_COMPONENT_PORT_INPUT_MESSAGE_ITERATOR_STATE_FINALIZING &&
+                               iterator->state != BT_SELF_COMPONENT_PORT_INPUT_MESSAGE_ITERATOR_STATE_FINALIZED) {
+                       BT_ASSERT_PRE_MSG("Message iterator is not being finalized or finalized: "
+                               "%!+i", iterator);
+                       ret = false;
+                       goto end;
+               }
+       }
+
+end:
+       return ret;
+}
+
 enum bt_self_component_port_status bt_self_component_port_remove_from_component(
                struct bt_self_component_port *self_port)
 {
@@ -145,7 +174,8 @@ enum bt_self_component_port_status bt_self_component_port_remove_from_component(
        struct bt_component *comp = NULL;
 
        BT_ASSERT_PRE_NON_NULL(port, "Port");
-
+       BT_ASSERT_PRE(port_connection_iterators_are_finalized(port),
+               "At least one message iterator using this port has the wrong state.");
        comp = (void *) bt_object_borrow_parent(&port->base);
        if (!comp) {
                BT_LIB_LOGV("Port already removed from its component: %!+p",
@@ -171,3 +201,33 @@ void *bt_self_component_port_get_data(const struct bt_self_component_port *port)
        BT_ASSERT_PRE_NON_NULL(port, "Port");
        return ((struct bt_port *) port)->user_data;
 }
+
+void bt_port_get_ref(const struct bt_port *port)
+{
+       bt_object_get_ref(port);
+}
+
+void bt_port_put_ref(const struct bt_port *port)
+{
+       bt_object_put_ref(port);
+}
+
+void bt_port_input_get_ref(const struct bt_port_input *port_input)
+{
+       bt_object_get_ref(port_input);
+}
+
+void bt_port_input_put_ref(const struct bt_port_input *port_input)
+{
+       bt_object_put_ref(port_input);
+}
+
+void bt_port_output_get_ref(const struct bt_port_output *port_output)
+{
+       bt_object_get_ref(port_output);
+}
+
+void bt_port_output_put_ref(const struct bt_port_output *port_output)
+{
+       bt_object_put_ref(port_output);
+}