weston-log-flight-rec: Don't allow more than one flight recorder to be
authorMarius Vlad <marius.vlad@collabora.com>
Wed, 7 Aug 2019 14:18:37 +0000 (17:18 +0300)
committerDaniel Stone <daniel@fooishbar.org>
Mon, 25 Nov 2019 20:29:49 +0000 (20:29 +0000)
created

Having a (single) global variable which others depend on it implies
having a single flight recorder present. Until we have a reason to
support multiple flight recorders limit the amount to a maximum of one.

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Reported-by: Pekka Paalanen <pekka.paalanen@collabora.com>
libweston/weston-log-flight-rec.c

index 1a20934038ca30a1d7ec77f7327cad19614b5128..17aca1487817a011c65fea75d8534225860fc81e 100644 (file)
@@ -233,6 +233,9 @@ weston_log_subscriber_create_flight_rec(size_t size)
        struct weston_debug_log_flight_recorder *flight_rec;
        char *weston_rb;
 
+       assert("Can't create more than one flight recorder." &&
+                       !weston_primary_flight_recorder_ring_buffer);
+
        flight_rec = zalloc(sizeof(*flight_rec));
        if (!flight_rec)
                return NULL;