libusb/serialInterface: Fixed a potential nullptr dereference issues …

…in the constructor
blackmagic-debug · Dec 10, 2023 · 50ed8fb · 50ed8fb
1 parent fc06643
commit 50ed8fb
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/libusb/serialInterface.cxx b/src/libusb/serialInterface.cxx
@@ -137,6 +137,11 @@ serialInterface_t::serialInterface_t(const usbDevice_t &usbDevice) : device{usbD
 		for (const auto epIndex : substrate::indexSequence_t{2U})
 		{
 			const auto endpoint{firstAltMode.endpoint(epIndex)};
+			if (!endpoint.valid())
+			{
+				console.error("Probe interface missing a requireed endpoint (index "sv, epIndex, ')');
+				return;
+			}
 			if (endpoint.direction() == endpointDir_t::controllerOut)
 				txEndpoint = endpoint.address();
 			else