From 6c5c4b73e0bae8b0a26201ebdd875bc62aab97cb Mon Sep 17 00:00:00 2001
From: "andrey.bychkov" <andrey@signalmodelling.ru>
Date: Wed, 5 Aug 2026 17:38:01 +0300
Subject: [PATCH] fix(PIClientServer): delete PIEthernet when client_factory
 returns nullptr

When the custom client_factory() returned nullptr, the incoming
PIEthernet pointer was leaked. Matches existing pattern used in the
max_clients overflow branch.
---
 libs/client_server/piclientserver_server.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libs/client_server/piclientserver_server.cpp b/libs/client_server/piclientserver_server.cpp
index edfd2083..aaea5d21 100644
--- a/libs/client_server/piclientserver_server.cpp
+++ b/libs/client_server/piclientserver_server.cpp
@@ -37,6 +37,7 @@ PIClientServer::Server::Server() {
 		auto sc = client_factory();
 		if (!sc) {
 			piCout << "ClientFactory returns nullptr!"_tr("PIClientServer");
+			delete c;
 			return;
 		}
 		sc->createForServer(this, c);