| Message ID | 20200824111837.Bluez.v3.1.I21d21871d85db48b07ba847742c7cb933024307c@changeid (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Luiz Von Dentz |
| Headers | show |
| Series | [Bluez,v3] adapter - Device needs to be in the temporary state after pairing failure | expand |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. While we are preparing for reviewing the patches, we found the following issue/warning. Test Result: checkgitlint Failed Outputs: 1: T1 Title exceeds max length (73>72): "adapter - Device needs to be in the temporary state after pairing failure" --- Regards, Linux Bluetooth
Hi Yu Liu, On Mon, Aug 24, 2020 at 11:30 AM Yu Liu <yudiliu@google.com> wrote: > > This caused the device hanging around as a discovered device forever > even if it is turned off or not in present. > > Reviewed-by: sonnysasaka@chromium.org > --- > > Changes in v3: > - Fix comment format > - Don't set temporary for trusted device > > Changes in v2: > - Fix title length and format issue > > Changes in v1: > - Initial change > > src/device.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/src/device.c b/src/device.c > index bb8e07e8f..103b5df27 100644 > --- a/src/device.c > +++ b/src/device.c > @@ -6008,6 +6008,14 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, > > if (status) { > device_cancel_authentication(device, TRUE); > + > + /* Put the device back to the temporary state so that it will be > + * treated as a newly discovered device. > + */ > + if (!device_is_paired(device, bdaddr_type) && > + !device_is_trusted(device)) > + btd_device_set_temporary(device, true); > + > device_bonding_failed(device, status); > return; > } > -- > 2.28.0.297.g1956fa8f8d-goog Applied, thanks.
diff --git a/src/device.c b/src/device.c index bb8e07e8f..103b5df27 100644 --- a/src/device.c +++ b/src/device.c @@ -6008,6 +6008,14 @@ void device_bonding_complete(struct btd_device *device, uint8_t bdaddr_type, if (status) { device_cancel_authentication(device, TRUE); + + /* Put the device back to the temporary state so that it will be + * treated as a newly discovered device. + */ + if (!device_is_paired(device, bdaddr_type) && + !device_is_trusted(device)) + btd_device_set_temporary(device, true); + device_bonding_failed(device, status); return; }
This caused the device hanging around as a discovered device forever even if it is turned off or not in present. Reviewed-by: sonnysasaka@chromium.org --- Changes in v3: - Fix comment format - Don't set temporary for trusted device Changes in v2: - Fix title length and format issue Changes in v1: - Initial change src/device.c | 8 ++++++++ 1 file changed, 8 insertions(+)