Hello,
I spotted that bluez has file profiles/audio/a2dp-codecs.h in its git
repository with some #if __BYTE_ORDER == checks, including #ifdef for
big endian systems.
But this file is fully broken for big endian systems. For example, there
is following structure which is same for both little and big endian
systems:
typedef struct {
uint32_t vendor_id;
uint16_t codec_id;
} __attribute__ ((packed)) a2dp_vendor_codec_t;
Which is valid only for little endian systems, as those definitions are
bound to little endian ordering -- not to host ordering.
Next there is a2dp_mpeg_t. It has different definition for little endian
and big endian systems.
little endian:
typedef struct {
uint8_t channel_mode:4;
uint8_t crc:1;
uint8_t layer:3;
uint8_t frequency:6;
uint8_t mpf:1;
uint8_t rfa:1;
uint16_t bitrate;
} __attribute__ ((packed)) a2dp_mpeg_t;
big endian:
typedef struct {
uint8_t layer:3;
uint8_t crc:1;
uint8_t channel_mode:4;
uint8_t rfa:1;
uint8_t mpf:1;
uint8_t frequency:6;
uint16_t bitrate;
} __attribute__ ((packed)) a2dp_mpeg_t;
But as you can see, bitrate on big endian definition is incorrect. It
is still in little endian. Proper way would be to define bitrate as
"uint8_t bitrate[2];" or as "uint8_t bitrate_lo; uint8_t bitrate_hi".
So basically this file a2dp-codecs.h is not compatible with big endian
systems, even there are "#elif __BYTE_ORDER == __BIG_ENDIAN" branches.
If you are not going to support big endian systems, I suggest you to
drop all those "#elif __BYTE_ORDER == __BIG_ENDIAN" parts as people
think that existence of these macros means that code is supported on big
endian systems.
It is better throw an error that big endian is broken as trying to
produce non-working binaries.
Moreover, some people started copying this a2dp-codecs.h file to other
projects which just leads to copy+paste of broken code.
--
Pali Rohár
pali.rohar@xxxxxxxxx
Attachment:
signature.asc
Description: PGP signature
