Tryanks/gousb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix a bug where EndpointInfo would always be picked from the last

Browse Source 
alternative setting.

When looking up matching EndpointInfo and InterfaceSetting for
an endpoint, a reference to range variable is stored. If range is
allowed to continue, the value of this variable will be updated.
It's a trick to allow use of "nil" pointer as a "found" variable, but it
requires a careful management of the range iterations.
This commit is contained in:
Sebastian Zagrodzki
2017-04-10 03:16:59 +02:00
parent b0d5519c67
commit 9cecdec977
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
usb/device.go
View File

@@ -132,6 +132,7 @@ func (d *Device) openEndpoint(cfgNum, ifNum, setNum, epAddr uint8) (*endpoint, e
setAlternate = i != 0 setAlternate = i != 0
debug.Printf("found setup: %+v [default: %v]\n", s, !setAlternate) debug.Printf("found setup: %+v [default: %v]\n", s, !setAlternate)
ifs = &s ifs = &s
break
} }
} }
if ifs == nil { if ifs == nil {
@@ -143,6 +144,7 @@ func (d *Device) openEndpoint(cfgNum, ifNum, setNum, epAddr uint8) (*endpoint, e
if endpointAddr(e.Number, e.Direction) == epAddr { if endpointAddr(e.Number, e.Direction) == epAddr {
debug.Printf("found ep #%d %s in %+v\n", e.Number, e.Direction, *ifs) debug.Printf("found ep #%d %s in %+v\n", e.Number, e.Direction, *ifs)
ep = &e ep = &e
break
} }
} }
if ep == nil { if ep == nil {