From 29639b98b3f36e25f484ef20e6edb660eac66d91 Mon Sep 17 00:00:00 2001
From: pstef <3462925+pstef@users.noreply.github.com>
Date: Wed, 19 Mar 2025 22:50:49 +0000
Subject: [PATCH] Use the return value of rbtree_insert to silence a warning

The function prototype wants us to be warned when the result is not
used, but it's not clear why - I assume to check for failure.
---
 ctr/ctr_linear.cpp | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/ctr/ctr_linear.cpp b/ctr/ctr_linear.cpp
index 1c906260a8..c8c0cb85a4 100644
--- a/ctr/ctr_linear.cpp
+++ b/ctr/ctr_linear.cpp
@@ -191,10 +191,13 @@ void* linearMemAlign(size_t len, size_t alignment)
 		sLinearPool.Deallocate(chunk);
 		return nullptr;
 	}
-	if (rbtree_insert(&sAddrMap, &node->node));
+	if (!rbtree_insert(&sAddrMap, &node->node)) {
+		sLinearPool.Deallocate(chunk);
+		return nullptr;
+	}
 
-   if (sLinearPool_maxaddr < (u32)sLinearPool.last->base)
-      sLinearPool_maxaddr = (u32)sLinearPool.last->base;
+	if (sLinearPool_maxaddr < (u32)sLinearPool.last->base)
+		sLinearPool_maxaddr = (u32)sLinearPool.last->base;
 
 	return chunk.addr;
 }