telemetry: fix lock leak on slot alloc failure

During allocation of a counter slot, spinlock isn't released in case of
failure. This could lead to deadlock in case of failed allocation. This
change fixes that by adding the missing release operation.

Signed-off-by: Wojciech Jablonski <wojciech.jablonski@intel.com>

Author: wjablon1

src/debug/telemetry/performance_monitor.c

@@ -138,8 +138,10 @@ struct perf_data_item_comp *perf_data_getnext(void)
 	 * ,and always set bit on bitmap alloc.
 	 */
 	ret = perf_bitmap_setbit(&performance_data_bitmap, idx);
-	if (ret < 0)
+	if (ret < 0) {
+		perf_bitmap_free(&performance_data_bitmap, idx);
 		return NULL;
+	}
 	return &perf_data[idx];
 }
 
@@ -451,18 +453,24 @@ static struct io_perf_data_item *io_perf_monitor_get_next_slot(struct io_perf_mo
 
 	ret = perf_bitmap_alloc(&self->io_performance_data_bitmap, &idx);
 	if (ret < 0)
-		return NULL;
+		goto out_unlock;
 	/* ref. FW did not set the bits, but here we do it to not have to use
 	 * isFree() check that the bitarray does not provide yet. Instead we will use isClear
 	 * ,and always set bit on bitmap alloc.
 	 */
 
 	ret = perf_bitmap_setbit(&self->io_performance_data_bitmap, idx);
-	if (ret < 0)
-		return NULL;
+	if (ret < 0) {
+		perf_bitmap_free(&self->io_performance_data_bitmap, idx);
+		goto out_unlock;
+	}
 
 	k_spin_unlock(&self->lock, key);
 	return &self->io_perf_data[idx];
+
+out_unlock:
+	k_spin_unlock(&self->lock, key);
+	return NULL;
 }
 
 int io_perf_monitor_release_slot(struct io_perf_data_item *item)