Fix no COM port #2325

Signed-off-by: Gareth Hancock <64541249+GazHank@users.noreply.github.com>

Author: GazHank

packages/bindings/src/serialport_win.cpp

@@ -792,8 +792,9 @@ void ListBaton::Execute() {
   char *name;
   char *manufacturer;
   char *locationId;
+  char *friendlyName;
   char serialNumber[MAX_REGISTRY_KEY_SIZE];
-  bool isCom = false;
+  bool isCom;
   while (true) {
     isCom = false;
     pnpId = NULL;
@@ -802,6 +803,8 @@ void ListBaton::Execute() {
     name = NULL;
     manufacturer = NULL;
     locationId = NULL;
+    friendlyName = NULL;
+    isCom = false;
 
     ZeroMemory(&deviceInfoData, sizeof(SP_DEVINFO_DATA));
     deviceInfoData.cbSize = sizeof(SP_DEVINFO_DATA);
@@ -836,6 +839,12 @@ void ListBaton::Execute() {
                                          sizeof(szBuffer), &dwSize)) {
       locationId = strdup(szBuffer);
     }
+    if (SetupDiGetDeviceRegistryProperty(hDevInfo, &deviceInfoData,
+                                         SPDRP_FRIENDLYNAME, &dwPropertyRegDataType,
+                                         reinterpret_cast<BYTE*>(szBuffer),
+                                         sizeof(szBuffer), &dwSize)) {
+      friendlyName = strdup(szBuffer);
+    }
     if (SetupDiGetDeviceRegistryProperty(hDevInfo, &deviceInfoData,
                                          SPDRP_MFG, &dwPropertyRegDataType,
                                          reinterpret_cast<BYTE*>(szBuffer),
@@ -867,6 +876,9 @@ void ListBaton::Execute() {
       if (locationId) {
         resultItem->locationId = locationId;
       }
+      if (friendlyName) {
+        resultItem->friendlyName = friendlyName;
+      }
       results.push_back(resultItem);
     }
     free(pnpId);

packages/bindings/src/serialport_win.h

@@ -54,6 +54,7 @@ struct ListResultItem {
   std::string serialNumber;
   std::string pnpId;
   std::string locationId;
+  std::string friendlyName;
   std::string vendorId;
   std::string productId;
 };
@@ -78,6 +79,7 @@ struct ListBaton : public Napi::AsyncWorker {
       setIfNotEmpty(item, "serialNumber", (*it)->serialNumber.c_str());
       setIfNotEmpty(item, "pnpId", (*it)->pnpId.c_str());
       setIfNotEmpty(item, "locationId", (*it)->locationId.c_str());
+      setIfNotEmpty(item, "friendlyName", (*it)->friendlyName.c_str());
       setIfNotEmpty(item, "vendorId", (*it)->vendorId.c_str());
       setIfNotEmpty(item, "productId", (*it)->productId.c_str());