autotools: LT_LANG([Windows Resource]) makes windres check redundant

Examples: add xusb sample

* Also improvements to WDK build scripts:
  cd to original dir on error, use 2 cores if available

VS2005: Bring x86_64 output directories in line with x86_32

Linux: fix inaccurate comment

* We don't stop on urbs/iso pkts with less data then requested

Linux: Remove unneeded status assignation

* status value was never used

Linux: Translate iso pkt status codes to libusb transfer status codes

* Linux negative errno from iso pkts status must be translated and reported.

Linux: Fix handling of urb status codes

* Adds EOVERFLOW handling in handle_control_completion
* Removes handling of codes that can never occur on an iso urb

Linux: Don't print errors when cancel_transfer fails with NOT_FOUND

* Under some cricumstances, LIBUSB_ERROR_NOT_FOUND is an expected
  return value for cancel_transfer so printing an error is undesirable.

Linux: Fix cancel_transfer return value when cancelling a multi-urb transfer

* if we've successfully cancelled the last urb, and then another urb
  fails with errno == EINVAL, we shouldn't return LIBUSB_ERROR_NOT_FOUND

libusbi.h: Add IS_EPIN() IS_EPOUT() IS_XFERIN() and IS_XFEROUT() macros

The macros make backend code a little simpler to read, since the
transfer and endpoint direction is checked in several places across
the supported platforms.

Add LIBUSB_TRANSFER_ADD_ZERO_PACKET flag to indicate need for ZLP

Some protocols which use USB require an extra zero length data packet
to signal end-of-transfer on bulk endpoints, if the last data packet
is exactly wMaxPacketSize bytes long.

This flag allows applications to inform libusb about this requirement,
so that libusb can handle the issue transparently.

At the moment the new flag is only supported on Linux, and submitting
a transfer with the flag set returns an error at submit time on other
systems. Hopefully implementations will soon follow for other systems.

References #6.

libusb.h: Add comma after last value in enums to ease future expansion

Linux: Rework bulk continuation check to generic kernel version check

The old check_flag_bulk_continuation() tested for one specific running
kernel version. The new kernel_version_ge() instead allows to test the
running kernel version against major, minor and sublevel parameters.

OpenBSD: Correctly catch attempts to do isochronous OUT transfers

LIBUSB_ENDPOINT_OUT is the value of the direction bit, which is 0 for
OUT transfers, so the previous condition could never evaluate to true.

Darwin: Do not reject devices with product id 0x0000

Fixes #128.

Do not call timerfd functions when timerfd is not being used

When libusb was built with timerfd support but used on a system
without timerfd support the library would hang indefinitely on
completion of the first transfer, since timerfd functions were
being called unconditionally and the error returned when timerfd
was not being used caused a confused internal state.

Many thanks to Ivo Smits for looking into the issue, proposing
an initial solution, and helping with testing! Fixes #73.

Windows: Fix MSVC6 IDE warning on line continuation in libusb-1.0.rc

* libusb\libusb-1.0.rc(21): Could not find the file LIBUSB_RC.
* only shows up first time after opening workspace.
* shows up on either build or clean.
* not actually due to rc.exe, but prior to it.
* probably an IDE bug.
* does not show up when running an exported makefile.

Signed-off-by: Michael Plante <michael.plante@gmail.com>

Windows: Add Renesas SuperSpeed support for libusb_get_device_speed()

* tests conducted using a Renesas PCIE USB 3.0 controller and driver and
  a mass storage USB 3.0 confirm that USB_NODE_CONNECTION_INFORMATION_EX
  uses (undocumented) value 3 for SuperSpeed

autogen.sh: Try to use libtoolize before trying glibtoolize

This helps on Mac OS X where an old glibtoolize is included in the
system and newer, manually installed, versions provide libtoolize.

See also http://marc.info/?m=132490560131894

Rename the lsusb example program to listdevs

This is intended to reduce confusion with the much more significant
lsusb utility which is part of the usbutils package.

core.c: Fix clang warning about useless store to a variable

Darwin: Clarify comments on event thread creation and termination

Darwin: Fix incorrect mutex protection of libusb_darwin_acfl global

Darwin: Use retained runloop variable instead of CFRunLoopGetCurrent()

Darwin: Fix libusb_init() and libusb_exit() refcount race condition

Use OSAtomicIncrement32Barrier() and OSAtomicDecrement32Barrier()
in darwin_init() and darwin_exit() to be thread safe.

threads_posix.c: Fix prototype warning by including threads_posix.h

Windows: Fix crash during enumeration with >= 64 connected devices

* indexes were outgrowing the unref_list array before realloc,
  resulting in out of bound access and crash.

Windows MSVC6: Remove SetupAPI, AdvAPI32 and OLE32 dependencies

* this ensures that libusb dependent applications only need
  to link with libusb on Windows
* (copied from Pete's msvc08 mods to msvc6 by Michael)

Windows: Output an error message on calls to libusb_get_pollfds()

Signed-off-by: Michael Plante <michael.plante@gmail.com>

io.c: Fix comment since pthreads isn't the only threading libusb uses

libusb.h: Cast in libusb_get_string_descriptor() to avoid MSVC6 warning

On MSVC6 bitwise OR promotes to int, causing the warning.

Windows: guid_to_string() is only discarded when macro-usbi_dbg used

* usbi_dbg encloses all references to guid_to_string
* MinGW/cygwin warn about an unused function, so the #if
  squelches this warning
* MSVC6 uses a variadic function form of usbi_dbg instead
  of a macro, so the compiler still "sees" guid_to_string
  and it therefore needs to always be defined for MSVC6,
  even if it's only a stub.
* So we define it if usbi_dbg is used OR if MSVC6 is used.

Signed-off-by: Michael Plante <michael.plante@gmail.com>

Windows: Fix undefined datatype 'LONG_PTR' in MSVC6

* issue reported by Elmi

Signed-off-by: Michael Plante <michael.plante@gmail.com>

Windows: Fix for MSVC6 preprocessor not accepting blank parameters

* unlike later iterations of Visual Studio, MSVC6 does not accept
  blank parameters on macro calls [eg. CALL(a, ,b)]
* blank params were used with the DLL_DECLARE and DLL_LOAD macros
* issue reported by Elmi

Windows: Switch MSVC6 .dsp project files from MBCS to Unicode

* MBCS (which is different from UTF-8) only makes sense if
  supporting Windows 95/98, which we don't
* (try to match Pete's vcproj changes in MSVC6)

Windows: Add missing MSVC6 lsusb.dsp project file

Signed-off-by: Michael Plante <michael.plante@gmail.com>

OpenBSD backend

configure.ac: Darwin: Move -lobjc from LIBS to PC_LIBS_PRIVATE

Since commit 40327cd134718475f6cec8935b856d4fdff2099c it is neccessary
to explicitly include -lobjc not only when linking libusb itself, but
also for programs linking statically against libusb. References #63.

See also http://marc.info/?m=132505900202378

examples: Link only with ../libusb/libusb-1.0.la and not with -lusb-1.0

Previous _LDFLAGS included both the freshly built libusb in ../libusb
and -lusb-1.0, where libtool would usually resolve the latter to an
already-installed libusb library in the system. The extra reference
to a second libusb library resulted in failure to build examples on
Mac OS X in some cases, and is plain wrong.

See also the thread at http://marc.info/?m=132637593623667

libusb/Makefile.am: Avoid recursive variables in .rc silent-rule

Not every make supports recursive variable expansion so some automake
versions complain about non-POSIX variable names ever since commit
70bec4a9f8ec28d36c731011fa24d37c74ad3523 which added support for
silent-rules in our rule to compile the Windows .rc file.

This commit removes the recursive variables and instead uses the
simple and generic GEN message and associated variable.

Core: Fix #122 warning: no previous prototype for ‘usbi_log_v’

Declare the usbi_log_v() function before using it.

Darwin: Set a name for the background thread on Mac OS X 10.6 and later

The call to pthread_setname_np() makes it easy to identify the
background thread in the Xcode debugger and in crash reports.

libusb/Makefile.am: Add libusb-1.0.rc dependency on version.h

libusb/Makefile.am: Make the libusb-1.0.rc rule support silent-rules

configure.ac: Enable libtool support for Windows Resource language

descriptor.c: Use size_t for minor 64 bit compliance improvement

Fixes #121.

Darwin: Fix warnings about mismatched signed/unsigned compare

References #121.

Fix unused variable warnings when without timerfd and/or when on Darwin

References #121.

Correctly handle LIBUSB_TRANSFER_OVERFLOW in libusb_control_transfer()

sync.c: In function `libusb_control_transfer':
sync.c:122: warning: enumeration value `LIBUSB_TRANSFER_OVERFLOW' not
handled in switch

Fixes #120.

Windows: do not set altsetting 0 as part of libusb_release_interface()

* pointed out by Travis Robinson and Xiaofan Chen
* similar to a change advised by Alan Stern for the Linux kernel:
  http://marc.info/?m=122790204410765

Windows: fix communication with hubs

* issue reported by René Haunstrup in http://marc.info/?m=130503019227814

Windows: add USB 3.0 controller support

* Windows 8, NEC/Renesas, TI, Fresco Logic, Etron, VIA, ASMedia
  (some of which untested!)
* includes workaround for NEC/Renesas USB 3.0 root hubs

Windows: add static keywords and drop duplicated or unneeded code

Windows: fix 'windows_error_str' defined but not used when not logging

libusbi.h: Silence unused context variable warnings when not logging

Windows: copy DLL file regardless of ddk_build.cmd dll argument case

Windows: Add uint32_t typedef for old MSC so that DDK can build lsusb

Darwin: fix port leak in darwin_clock_gettime()

Darwin: remove unnecessary run loop invalidation during libusb_exit()

Darwin: fix port leak during libusb_exit()

Darwin: fix potential leak on libusb_claim_interface() error

Darwin: Fix #117 transfer size 64/32 bit issue for transfer callbacks

The transfer size is now always truncated to 32 bits.

Remove username from SF FRS rsync upload command

Support release candidate versions in configure.ac and libusb-1.0.rc

Tidy .gitignore a little

There's no xusb, we need a glob pattern for matching .exe files, and
fpusb tarballs aren't very common in the libusb source dir.

Docs: Clarify that libusb_handle_events_timeout() tv param can't be NULL

The example code and API doc for libusb_handle_events_timeout() could be
interpreted as it being OK to pass a NULL pointer for the tv argument (I
interpreted it like that when I first started coding for libusb).

This patch changes the docs to make it clear that one must always supply
a tv struct to libusb_handle_events_timeout.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Makefile.am: Do not remove ChangeLog on make distclean, fixes #116

[pbatard: Remove it on make maintainer-clean instead]

Darwin: Reduce race likelihood between cancellation and device disconnect

References #88. The race condition still remains, but this change
makes it less likely to trigger.

Document that libusb_get_device_list() can return any libusb_error code

Fixes #87.

Linux: Fix #81 URB double free race condition on device disconnect

A submitted transfer that has just been failed by the kernel could be
picked up by an event handler to be cleaned up, where freeing of URB
memory would race with the submit function doing it's own cleanup and
freeing as a result of the submit failing.

libusb_submit_transfer() always holds itransfer->lock, so the race can
be avoided by taking that lock also in the cleanup path and checking
that the URB memory has not already been freed before freeing it there.

As http://libusb.org/ticket/81#comment:14 notes there is still another
possible, but unlikely, race condition between libusb_submit_transfer()
and an event handling thread. That will require more work to solve.

[stuge: Add check in cleanup path that URBs have not already been freed]

Add libusb_error_name() API function

Until we have i18ned error messages we can at least offer easy
translation from numeric error code to the error code's name.

[hansg: switch() on enum to get a compiler warning if a case is missing]
[hansg: Better return value when called with an unknown error code]
[hansg: Fix DEFAULT_VISIBILITY in definition to compile on all platforms]

Add libusb_has_capability() API function

Since it is currently not planned to change the filename of the libusb
library between libusb-1.0 versions at least on Windows systems it's
important to have a capability API.

Currently there exists only one capability; LIBUSB_CAP_HAS_CAPABILITY,
which tests support for the libusb_has_capability() API.

Fix typos in introductory documentation

Linux: Add libusb_open() debug and error messages with the device path

Fixes #109.

Add debug message with libusb_open() error code

References #109.

Document libusb_handle_events_completed() and _timeout_completed()

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
[stuge: Note that the old racy functions should be avoided by new code]

Fix #56 race condition causing delayed completion of sync transfers

The sync API had a race where it would check a condition to know if it
needed to call a libusb_handle_events() function. However, the check
was done outside of the lock that is held while the condition is set,
so another thread could completely serve whatever was needed to make
the condition true between it being checked and the event handler being
called. This situation would be detected after a libusb-internal timeout
of 60 seconds, after which the transfer would be completed without
error, but with significant delay.

Original patch at http://marc.info/?l=libusb-devel&m=127252114815709

Changes by Hans de Goede:
- Renamed the "race-proof" functions from libusb_handle_events*_check()
  to libusb_handle_events*_completed()
- Drop r = 0 setting in libusb_handle_events_timeout_completed()
  (to make both completed checking cases identical flow wise)

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
[stuge: Simplify libusb_handle_events_timeout() change with a goto]
[pbatard: Fix _handle_events_timeout() and _completed() definitions]

Darwin: Use a condition to signal async thread ready in libusb_init()

Fix #64 use of reserved identifiers throughout libusb

Darwin: fix libusb_get_device_list() and libusb_claim_interface() leaks

Darwin: use kIOMasterPortDefault instead of creating a master port

Allow devices with zero configurations to be discovered

At least in Linux, wireless USB devices have zero configurations until
they have been authorized. This device state can be set for any USB
device using:

echo 0 > /sys/bus/usb/devices/?-?/authorized

An unauthorized device would previously make usbi_sanitize_device()
return LIBUSB_ERROR_IO, although there really was no I/O error.

Reported-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>

Linux: Continue enumeration even if one device returns an error

Linux: Handle single- and two-digit kernel versions like 3.0 and 4

The code has been tested with various version strings.

[stuge: Remove bogus string length check and optimize for newer kernels]

Windows: Provide libusb_get_device_speed() data

* use (currently superspeed unaware) USB_NODE_CONNECTION_INFORMATION_EX

Fixes #45.

[stuge: Explicitly convert OS device speeds to libusb device speeds]
[pbatard: Add breaks overlooked when adding explicit conversion]

Darwin: Provide libusb_get_device_speed() data

References #45.

Linux: Provide libusb_get_device_speed() data from sysfs

References #45.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Add new API function libusb_get_device_speed()

Reworked version of my earlier patch, largely modelled after (if not
copy and pasted from) Trygve Laugstøl's <trygvis@inamo.no> similar
commit 5a6541e0d80fb1f21e2b960bc2337a612f9d74fb in
git://git.libusb.org/libusb-trygvis.git
http://git.libusb.org/libusb-trygvis.git
http://git.libusb.org/?p=libusb-trygvis.git;a=commitdiff;h=5a6541e0d

This patch does not add any OS specific code. The supporting code in
each backend follows in separate patches. References #45.

Signed-off-By: Hans de Goede <hdegoede@redhat.com>

Darwin: Improve device enumeration performance

The device location is saved, and later used whenever iterating over
devices. Fixes #111 and #112.

[stuge: Formatting fixes and split out libusb_get_device_speed() change]
[stuge: Combine with patch that uses the saved device location]

Linux: Use __read_sysfs_attr() to get busnum and devaddr

Remove code duplication.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Linux: Add __read_sysfs_attr() helper function

On Linux we often need to read (postive) integers from sysfs. This patch
adds a helper function for this. This is a preparation patch for adding
a libusb_get_device_speed() function to libusb.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Linux: Stop kernel from re-attaching in-kernel driver after reset

When an interface is bound to the usbfs driver (iow claimed), the
kernel will unbind it, and then after the reset do a device_attach
on the interface, which will bind the default in kernel driver to
the interface.

So if an app has detached the in kernel driver, and claimed the
interface and then does a libusb_reset_device. Things end up with
the interface no longer being bound to the usbfs driver (so no longer
claimed) and instead it is bound to the in kernel driver (iow the in
kernel driver is re-attached).

We can stop this from happening by releasing all claimed interfaces
before the reset, as the kernel will not do the device attach after
reset, if no driver was bound to the interface before the reset.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Core: Use USB_MAXINTERFACES rather then sizeof(claimed_interfaces)

claimed_interfaces is a long and thus its size differs on 32
versus 64 bits, this patch uses USB_MAXINTERFACES instead of
sizeof(claimed_interfaces) on interface number range checks
for consistent behavior independent of the arch.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Linux: Fix warnings seen when compiling with gcc-4.6

Compiling with gcc-4.6 results in the following warnings:
os/linux_usbfs.c: In function 'op_get_configuration':
os/linux_usbfs.c:1144:6: warning: variable 'r' set but not used ...
os/linux_usbfs.c: In function 'op_handle_events':
os/linux_usbfs.c:2091:40: warning: 'status' may be used uninitialized ...
os/linux_usbfs.c:2044:6: note: 'status' was declared here
dpfp.c: In function 'save_to_file':
dpfp.c:228:9: warning: variable 'ignore' set but not used ...
dpfp_threaded.c: In function 'save_to_file':
dpfp_threaded.c:257:9: warning: variable 'ignore' set but not used ...

This patch fixes these. (The second comes from handle_control_completion()
which gcc seems to inline into op_handle_events().)

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
[stuge: Add fixes for dpfp examples and update source references]

Darwin: Fix #108 crash when a device is unplugged

libusb.h: Add missing LIBUSB_CLASS_ codes currently defined by usb.org

LIBUSB_CLASS_PHYSICAL
LIBUSB_CLASS_SMART_CARD
LIBUSB_CLASS_CONTENT_SECURITY
LIBUSB_CLASS_VIDEO
LIBUSB_CLASS_PERSONAL_HEALTHCARE
LIBUSB_CLASS_DIAGNOSTIC_DEVICE

Per http://www.usb.org/developers/defined_class

libusb.h: Add LIBUSB_CLASS_IMAGE in addition to LIBUSB_CLASS_PTP

The old LIBUSB_CLASS_PTP name is kept for compatibility. Fixes #106.

libusb.h: Fix LIBUSB_CLASS_PRINTER description typo

References #106.

Darwin: Fix #100 warning cast to pointer from integer of different size

configure.ac: Fix #97 clang warning about -fgnu89-inline

Remove the test that sets the flag since GNU89 inline semantics are not
required by libusb.

[stuge: Also remove reference to the test result]