diff options
| author | Dean Camera <dean@fourwalledcubicle.com> | 2011-05-14 02:17:58 +0000 |
|---|---|---|
| committer | Dean Camera <dean@fourwalledcubicle.com> | 2011-05-14 02:17:58 +0000 |
| commit | 049e9309637cd057689cf273b2023afb0d6876ae (patch) | |
| tree | 184919dd3e728aad41bc3bb1567b6f95f18564aa /LUFA/Drivers/USB/Core/AVR8 | |
| parent | 78e58b6d1c645c3ba1afaa536e2ba5bdab2b235b (diff) | |
| download | lufa-049e9309637cd057689cf273b2023afb0d6876ae.tar.gz lufa-049e9309637cd057689cf273b2023afb0d6876ae.tar.bz2 lufa-049e9309637cd057689cf273b2023afb0d6876ae.zip | |
Fixed possible invalid program execution when in host mode if corrupt descriptor lengths are supplied by the attached device.
Minor code cleanups to add const and reformat where missing, as well as abstract out the internal device signature start address into a macro, so that it can be altered to suit particular devices within a single architecture if needed.
Add missing documentation to the USB_Device_States_t enum.
Diffstat (limited to 'LUFA/Drivers/USB/Core/AVR8')
| -rw-r--r-- | LUFA/Drivers/USB/Core/AVR8/Device_AVR8.h | 22 |
1 files changed, 15 insertions, 7 deletions
diff --git a/LUFA/Drivers/USB/Core/AVR8/Device_AVR8.h b/LUFA/Drivers/USB/Core/AVR8/Device_AVR8.h index f88c2f1b2..c98e17cd5 100644 --- a/LUFA/Drivers/USB/Core/AVR8/Device_AVR8.h +++ b/LUFA/Drivers/USB/Core/AVR8/Device_AVR8.h @@ -99,16 +99,22 @@ * On unsupported devices, this will evaluate to \ref NO_DESCRIPTOR and so will force the host to create a pseudo-serial * number for the device. */ - #define USE_INTERNAL_SERIAL 0xDC + #define USE_INTERNAL_SERIAL 0xDC /** Length of the device's unique internal serial number, in bits, if present on the selected microcontroller * model. */ - #define INTERNAL_SERIAL_LENGTH_BITS 80 + #define INTERNAL_SERIAL_LENGTH_BITS 80 + + /** Start address of the internal serial number, in the appropriate address space, if present on the selected microcontroller + * model. + */ + #define INTERNAL_SERIAL_START_ADDRESS 0x0E #else - #define USE_INTERNAL_SERIAL NO_DESCRIPTOR + #define USE_INTERNAL_SERIAL NO_DESCRIPTOR - #define INTERNAL_SERIAL_LENGTH_BITS 0 + #define INTERNAL_SERIAL_LENGTH_BITS 0 + #define INTERNAL_SERIAL_START_ADDRESS 0 #endif /* Function Prototypes: */ @@ -198,12 +204,13 @@ return (UDADDR & (1 << ADDEN)); } - static inline void USB_Device_GetSerialString(uint16_t* UnicodeString) + #if (USE_INTERNAL_SERIAL != NO_DESCRIPTOR) + static inline void USB_Device_GetSerialString(uint16_t* const UnicodeString) { uint_reg_t CurrentGlobalInt = GetGlobalInterruptMask(); GlobalInterruptDisable(); - uint8_t SigReadAddress = 0x0E; + uint8_t SigReadAddress = INTERNAL_SERIAL_START_ADDRESS; for (uint8_t SerialCharNum = 0; SerialCharNum < (INTERNAL_SERIAL_LENGTH_BITS / 4); SerialCharNum++) { @@ -223,7 +230,8 @@ SetGlobalInterruptMask(CurrentGlobalInt); } - + #endif + #endif #endif |