From 326ecdf238921dc12645f2c31f4882499a387aa0 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Wed, 16 Dec 2020 16:54:00 +0100 Subject: Add assertions for atom buffer sizes This avoids warnings about potential null pointer dereferences, which is true, but in this case the buffer are static so it's really a programming error if there is ever an overrun. --- plugins/eg-scope.lv2/examploscope_ui.c | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'plugins/eg-scope.lv2/examploscope_ui.c') diff --git a/plugins/eg-scope.lv2/examploscope_ui.c b/plugins/eg-scope.lv2/examploscope_ui.c index 48ffec4..32584a4 100644 --- a/plugins/eg-scope.lv2/examploscope_ui.c +++ b/plugins/eg-scope.lv2/examploscope_ui.c @@ -105,6 +105,8 @@ send_ui_state(LV2UI_Handle handle) LV2_Atom* msg = (LV2_Atom*)lv2_atom_forge_object( &ui->forge, &frame, 0, ui->uris.ui_State); + assert(msg); + // msg[samples-per-pixel] = integer lv2_atom_forge_key(&ui->forge, ui->uris.ui_spp); lv2_atom_forge_int(&ui->forge, ui->stride); @@ -137,6 +139,9 @@ send_ui_disable(LV2UI_Handle handle) LV2_Atom_Forge_Frame frame; LV2_Atom* msg = (LV2_Atom*)lv2_atom_forge_object( &ui->forge, &frame, 0, ui->uris.ui_Off); + + assert(msg); + lv2_atom_forge_pop(&ui->forge, &frame); ui->write(ui->controller, 0, @@ -161,6 +166,9 @@ send_ui_enable(LV2UI_Handle handle) LV2_Atom_Forge_Frame frame; LV2_Atom* msg = (LV2_Atom*)lv2_atom_forge_object( &ui->forge, &frame, 0, ui->uris.ui_On); + + assert(msg); + lv2_atom_forge_pop(&ui->forge, &frame); ui->write(ui->controller, 0, -- cgit v1.2.1