fix: use productID to detect ledger type

fixes issue on certain linux hosts when strings are not
decoded properly. though strings issue must be fixed,
it is still remains convenient to ProductID as source
of detection Ledger type

Signed-off-by: Artur Troian <[email protected]>

Author: troian

ledger_hid.go

@@ -29,9 +29,8 @@ import (
 const (
 	VendorLedger         = 0x2c97
 	UsagePageLedgerNanoS = 0xffa0
-	//ProductNano     = 1
-	Channel    = 0x0101
-	PacketSize = 64
+	Channel              = 0x0101
+	PacketSize           = 64
 )
 
 type LedgerAdminHID struct{}
@@ -42,6 +41,12 @@ type LedgerDeviceHID struct {
 	readChannel chan []byte
 }
 
+// list of supported product ids as well as their corresponding interfaces
+var supportedLedgerProductID = map[uint16]int{
+	0x4011: 0, // Ledger Nano X
+	0x1011: 0, // Ledger Nano S
+}
+
 func NewLedgerAdmin() *LedgerAdminHID {
 	return &LedgerAdminHID{}
 }
@@ -72,9 +77,11 @@ func (admin *LedgerAdminHID) ListDevices() ([]string, error) {
 func isLedgerDevice(d hid.DeviceInfo) bool {
 	deviceFound := d.UsagePage == UsagePageLedgerNanoS
 	// Workarounds for possible empty usage pages
-	return deviceFound ||
-		(d.Product == "Nano S" && d.Interface == 0) ||
-		(d.Product == "Nano X" && d.Interface == 0)
+	if interfaceID, supported := supportedLedgerProductID[d.ProductID]; deviceFound || (supported && (interfaceID == d.Interface)) {
+		return true
+	}
+
+	return false
 }
 
 func (admin *LedgerAdminHID) CountDevices() int {
@@ -92,8 +99,8 @@ func (admin *LedgerAdminHID) CountDevices() int {
 
 func newDevice(dev *hid.Device) *LedgerDeviceHID {
 	return &LedgerDeviceHID{
-		device: dev,
-		readCo: new(sync.Once),
+		device:      dev,
+		readCo:      new(sync.Once),
 		readChannel: make(chan []byte),
 	}
 }