Libusb1 Async Control Buffer Null about python-libusb1 HOT 4 CLOSED

The idea behind the change in #21 is that __transfer_py_buffer gets modified when the memory behind __transfer_buffer is modified inside the C library, to avoid copies.

But I broke this with this line in setControl, which is likely the cause of this bug:

self.__transfer_py_buffer = transfer_py_buffer[CONTROL_SETUP_SIZE:]

Could you change this line to the following and report if it fixes the problem ?

self.__transfer_py_buffer = memoryview(transfer_py_buffer)[CONTROL_SETUP_SIZE:]

The problem with this fix is that, while python3's bytearray only accepts integers for item assignment, python2's memoryview rejects integers (I'm surprised to be apparently the first to report this on python's bugtracker given the number of google hits this issue has...). So if I apply it, I force the user of my module to be actively incompatible... Gah.

from python-libusb1.

Awesome, this solved the problem.
Yeah, this is a pretty amazing package and has been incredible useful in my work, so keep up the great work! Also thanks for the quick response!

from python-libusb1.

Thanks for the warm words, and happy my work is useful to you !

I reopen this issue, as the bug is still present in master - just so I do not forget to push the fix once I know what to expect from python 2.7.

from python-libusb1.

Now fixed in master, and released on pypi as 1.6.4, closing.

from python-libusb1.