| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
* Please update your URL in .git/config or issue a new clone
from: git://github.com/libusbx/libusbx.git
* The libusbx team apologizes for any inconvenience caused
|
|
|
|
|
|
|
|
|
| |
* The LOG_LEVEL_ enums, that were moved to the public API
in 933a319469bcccc962031c989e39d9d1f44f2885 may conflict
with applications/headers that also define their own
LOG_LEVEL_ values internally.
* As a matter of fact, as per Trac #31, this produces a
conflict with libusb-compat, as it defines its own levels.
|
|
|
|
|
|
|
|
|
|
| |
* http://sourceforge.net/mailarchive/message.php?msg_id=29418038
* core.c:700:4: warning: Function call argument is an uninitialized value
* darwin_usb.c:1713:11: warning: Access to field 'cfSource' results in a
dereference of a null pointer (loaded from variable 'hpriv')
* sync.c/dpfp.c/dpfp_threaded.c: warning: Result of 'malloc' is converted
to a pointer of type 'unsigned char', which is incompatible with sizeof
operand type
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* The library is now compiled with warning level 4 for VS2010
* Move silencing of 4200, 28125 and 28719 to msvc/config.h
* Add fixes in core to silence unused variables warnings
* Ensure that spinlock is always set interlocked in poll_windows
* Add missing check for calloc return value
* Fix data assignation in conditionals warnings
* Fix an OACR/Prefast error related to the use of strncpy in xusb.c
* Also fixes whitespace inconsistencies in core
* Issues reported by Orin Eman and Xiaofan Chen. See:
https://sourceforge.net/mailarchive/message.php?msg_id=29412656
|
|
|
|
|
|
|
| |
* Even though the library name is optional as specified by Microsoft,
some recent versions of libtool require one in libusb-1.0.def.
* Reference thread in MinGW-w64 mailing list.
http://comments.gmane.org/gmane.comp.gnu.mingw.w64.general/5141
|
|
|
|
|
|
|
|
|
| |
* windows_usb.c(161): warning #181: argument is incompatible
with corresponding format string conversion
* windows_usb.c(2124): warning #111: statement is unreachable
usbi_dbg("ERROR: broken timer thread");
* multiple: warning #188: enumerated type mixed with another
* Issues reported by Orin Eman
|
| |
|
| |
|
|
|
|
|
|
|
| |
* Result of 'calloc' is converted to a pointer of type 'unsigned char *',
which is incompatible with sizeof operand type 'PUSB_CONFIGURATION_DESCRIPTOR'
* priv->config_descriptor is indeed an array of pointers, with each descriptor
allocated, rather than a sequential list of fixed descriptor.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
core.c:
* Result of 'malloc' is converted to a pointer of type 'struct libusb_device *',
which is incompatible with sizeof operand type 'void *'
* Memory is never released; potential leak of memory pointed to by 'devs'
* Assigned value is garbage or undefined (due to potentially empty and
uninitialized device list)
descriptor.c:
* Function call argument is an uninitialized value
io.c:
* Call to 'malloc' has an allocation size of 0 bytes
* Branch condition evaluates to a garbage value (due to get_next_timeout
returning a negative error code instead of zero on error)
|
| |
|
|
|
|
|
| |
* Also add version_nano.h to EXTRA_DIST as it is not added to the
dist archive by default
|
|
|
|
|
|
|
|
|
|
| |
* Because we use the WINAPI calling convention, the def file MUST have the @n
aliases. There is no way around this as MinGW's .o use decoration always
for __stdcall, which can't be turned off.
* dlltool must therefore be invoked to create a proper import lib from the .def,
using the --kill-at option.
* To do that, a CREATE_IMPORT_LIB autotools variable is introduced.
* Note: the .def file is currently maintained manually.
|
|
|
|
|
|
|
|
| |
* p = realloc(p, new_size) does not free the original buffer in case of
a realloc failure.
* reallocf() can be used to do so, but is not available on all platforms.
* This patch introduces usbi_reallocf() in libusbi.h and use that instead of realloc
* Issue and original patch submitted by Moritz Lipp (trac #27)
|
|
|
|
|
|
|
|
| |
* With gcc-4.6 the option -fomit-frame-pointer is turned on per default
even for the win32 target. This results in segfaults for any application
using setjmp/longjmp and MSVCRT.dll on WinXP.
See http://gcc.gnu.org/ml/gcc/2011-10/msg00351.html
* Issue reported by Lars Kanis
|
| |
|
|
|
|
|
|
| |
* Fix naked "exit", which no exit code, to prevent a build failure
from being noticed automatically if a build dependency is lacking.
* Also printed error message to stderr rather than stdout.
|
|
|
|
|
|
|
|
|
|
|
| |
* On some Linux platforms, libusbx compilation breaks with:
error: implicit declaration of function ‘pthread_mutexattr_settype’
* This regression, introduced in 463dda06db5da5de0eab32820c7af60605625afe,
is due to pthread.h needing __USE_UNIX98, which is tied to _XOPEN_SOURCE
or _GNU_SOURCE being correctly defined, and which the inclusion of
<unistd.h> before the _XOPEN_SOURCE override modified
* As _GNU_LINUX ensures the definition of __USE_UNIX98 and we require it
for syscalls, we now only define _GNU_SOURCE for Linux.
|
|
|
|
|
|
| |
* Because of missing [], "LIBUSB_MICROLIBUSB_RC" was being displayed
in lieu of version number
* Regression introduced with c9d41fe5f0324d969f758f42843e1fd05859f438
|
|
|
|
|
| |
* Also fix a missing space in debug log messages introduced
with the previous timestamp logging update
|
| |
|
| |
|
|
|
|
|
| |
* Not applied: AC_PREREQ([2.68]) as this doesn't seem
necessary and 2.68 was released less than 2 years ago.
|
|
|
|
| |
* This effectively removes the use of stdout for info messages
|
|
|
|
|
|
| |
* Further alignment of log messages according to severity
* Fix "'darwin_error_str' defined but not used" warning when
option --disable-log is used (reported by Xiaofan Chen)
|
|
|
|
|
| |
* Generated from a MinGW32 generated DLL using the command:
echo -e "LIBRARY\nEXPORTS" > libusb-1.0.def; strings libusb-1.0.dll | grep libusb | grep @ | sort | sed -e "s/\(.*\)@\([0-9]*\)/ \1\n \1@\2 = \1/" >> libusb-1.0.def
|
| |
|
| |
|
|
|
|
| |
* Also update xusb sample to use these levels
|
|
|
|
|
| |
* Some informational messages were actually debug or warning
* Fixes #1
|
|
|
|
|
|
| |
* Adds libusb_get_port_number, libusb_get_parent and libusb_get_port_path
* Linux implementation provided by Alan Stern, OS X by Nathan Hjelm
* Unsupported for *BSD platforms
|
|
|
|
| |
* Also remove extra lines at the end of samples
|
|
|
|
|
|
|
|
| |
* Failure to read the config descriptor through regular API calls can
occur for some hubs (eg. 05E3:0608) and is of no major consequence.
* Later failures still reported with error severity as they may hide
something more dramatic
* Issue reported by Jerry from NovaDSP
|
|
|
|
| |
* Cygwin uses POSIX threads, not Windows threads
|
| |
|
|
|
|
|
|
|
|
|
| |
* As version.h processed by autotools, the automatic updating of the
nano there can result in unneeded reconfs (eg. after issuing a git
pull, regardless of whether the conf is still valid)
* Avoid this by placing the nano in its own file, as it won't be
processed by autoconf. This also simplifies the versioning git hooks
and removes the sed dependency.
|
|
|
|
| |
* By default automake looks for a source file foo.c for a foo binary.
|
|
|
|
| |
* Similar to libusb patch a129732fb45fc424743f26f01c906e4514f11521
|
|
|
|
|
|
| |
* While both should work, __linux__ seems to be preferred and
what we use in threads_posix.c
* Also fix a typo in .private/README.txt
|
| |
|
|
|
|
|
|
|
|
|
| |
* num_iso_packets was a "size_t" in "struct darwin_transfer_priv" and an
"int" in "struct libusb_transfer". The field is now an int in the two
structures
* Fixes warning
os/darwin_usb.c: In function ‘submit_iso_transfer’:
os/darwin_usb.c:1334: warning: comparison between signed and unsigned
|
|
|
|
|
|
|
|
|
| |
* Return the more appropriate LIBUSB_ERROR_IO instead of
LIBUSB_ERROR_OTHER for the cases LIBUSB_TRANSFER_ERROR and
LIBUSB_TRANSFER_CANCELLED
* Fixes warnings similar to the following when using the GCC
option -Wswitch-enum:
warning: enumeration value ‘LIBUSB_TRANSFER_ERROR’ not handled in switch
|
|
|
|
|
|
| |
* Uses syscall(SYS_getthrid) which requires real thread support,
currently only available in 5.1-current (but not 5.1-release).
For OpenBSD <= 5.1, -1 will be returned for the thread ID.
|
|
|
|
|
| |
* Without this, OpenBSD produces the following warning:
strcat() is almost always misused, please use strlcat()
|
| |
|
| |
|
|
|
|
| |
* "$@" will correctly handle arguments with spaces
|
|
|
|
| |
* The script will not continue if something fails (like a command not found)
|
|
|
|
|
|
| |
* Script is valid for bash but not any /bin/sh
* Thanks to Xiaofan Chen for the bug report
http://sourceforge.net/mailarchive/message.php?msg_id=29217871
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Without this change the Windows backend needed to call usbi_fd_notification()
from within the backend's submit_transfer. This can cause deadlock when
attempting to lock the event lock if another thread was processing events on
the just-submitted transfer.
The deadlock comes about as the thread calling libusb_submit_transfer acquires
the transfer mutex before trying to acquire the event lock; this is the other
order of lock acquisition from an event thread handling activity on the just
submitted transfer. This could lead to one of two deadlocks:
1) If the transfer completes while usbi_fd_notification() is waiting for
the event lock and the callback attempts to resubmit the transfer.
2) If the transfer timeout is hit while usbi_fd_notification() is waiting
for the event lock then the attempt to cancel the transfer will deadlock.
This patch fixes both of these deadlocks by having libusb_submit_transfer()
only call usbi_fd_notification() after having released the transfer mutex.
|