Re: [PATCH BlueZ 5/5] monitor: Add parsing Service Data of UUID 128

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Paul


On Sat, 29 Jul 2023 at 06:42, Paul Menzel <pmenzel@xxxxxxxxxxxxx> wrote:
>
> Dear Łukasz,
>
>
> Thank you for your patch.
>
> Am 29.07.23 um 02:45 schrieb Łukasz Rymanowski:
> > In the advertising report, instead of deciding EIR 0x21 into this:
>
> Do you mean deciphering?

Thanks for pointing it out. Just sent v2 for this patch.

>
> >
> >       Unknown EIR field 0x21: 3a048f2ba97ea6ae1b49c619e973a4a701480d130d
> >
> > We can do bit better
> >
> >          Service Data UUID 128: Vendor specific
> >            Data: 01480d130d
> > ---
> >   monitor/packet.c | 12 ++++++++++++
> >   1 file changed, 12 insertions(+)
> >
> > diff --git a/monitor/packet.c b/monitor/packet.c
> > index 3031b028f..21e08ccef 100644
> > --- a/monitor/packet.c
> > +++ b/monitor/packet.c
> > @@ -3997,6 +3997,18 @@ static void print_eir(const uint8_t *eir, uint8_t eir_len, bool le)
> >                       print_service_data(data, data_len);
> >                       break;
> >
> > +             case BT_EIR_SERVICE_DATA128:
> > +                     if (data_len <= 16)
> > +                             break;
> > +
> > +                     print_field("Service Data UUID 128: %s ",
> > +                                             bt_uuid128_to_str(&data[0]));
> > +
> > +                     if (data_len > 16)
> > +                             print_hex_field("  Data", &data[16],
> > +                                                             data_len - 16);
> > +
> > +                     break;
> >               case BT_EIR_RANDOM_ADDRESS:
> >                       if (data_len < 6)
> >                               break;
>
> Reviewed-by: Paul Menzel <pmenzel@xxxxxxxxxxxxx>
>
>
> Kind regards,
>
> Paul

Best
Łukasz




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux