On Fri, 2019-07-05 at 15:02 +0200, Pali Rohár wrote: > Rename struct rtp_payload to rtp_sbc_payload as it is specific for SBC > codec payload. > > Add proper checks for endianity in rtp.h header and use uint8_t type > where appropriated. > > Field frame_count is only 4 bit number, so add checks to prevent overflow. > > And because is_fragmented field is not parsed by decoder there is no > support for decoding fragmented SBC frames. So throw an error in this case. > --- > src/modules/bluetooth/a2dp-codec-sbc.c | 35 ++++++++++++++------ > src/modules/bluetooth/rtp.h | 58 +++++++++++++++++++--------------- > 2 files changed, 58 insertions(+), 35 deletions(-) Looks good to me. -- Tanu https://www.patreon.com/tanuk https://liberapay.com/tanuk _______________________________________________ pulseaudio-discuss mailing list pulseaudio-discuss@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
