Skip to content

Conversation

@LingaoM
Copy link
Contributor

@LingaoM LingaoM commented Nov 3, 2021

When Device use pb-gatt bearer provisioning, and first
provisioning failed due to some reasons(invalid oob etc..).

Which cause proxy:gatt_disconnectd incorrected write conn_count

Cause this var euqals -1.

Signed-off-by: Lingao Meng [email protected]

When Device use pb-gatt bearer provisioning, and first
provisioning failed due to some reasons(invalid oob etc..).

Which cause `proxy:gatt_disconnectd` incorrected write `conn_count`

Cause this var euqals `-1`.

Signed-off-by: Lingao Meng <[email protected]>
@LingaoM LingaoM added the bug The issue is a bug, or the PR is fixing a bug label Nov 3, 2021
Copy link
Contributor

@trond-snekvik trond-snekvik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can I trigger this issue? By connecting to a mesh node without using mesh proxy?

@LingaoM
Copy link
Contributor Author

LingaoM commented Nov 3, 2021

How can I trigger this issue? By connecting to a mesh node without using mesh proxy?

While i’m use mesh_shell:

  • mesh init
  • mesh pb-gatt on
  • use nRF Mesh App Provision use Invalid OOB.
  • provision failed
  • mesh pb-gatt on
  • use nRF Mesh App Provision use Valid OOB.

This problem will be occur.

@cfriedt
Copy link
Member

cfriedt commented Nov 4, 2021

Is this also a bug that exists in LTS @jhedberg ? If so, we should add the backport label and create an issue.

@jhedberg
Copy link
Member

jhedberg commented Nov 4, 2021

Is this also a bug that exists in LTS @jhedberg ? If so, we should add the backport label and create an issue.

@cfriedt agreed.

@trond-snekvik @LingaoM if this issue also exists in LTS, could you take the appropriate action as outlined by @cfriedt ?

@LingaoM
Copy link
Contributor Author

LingaoM commented Nov 5, 2021

Is this also a bug that exists in LTS @jhedberg ? If so, we should add the backport label and create an issue.

@cfriedt agreed.

@trond-snekvik @LingaoM if this issue also exists in LTS, could you take the appropriate action as outlined by @cfriedt ?

While i'm deep in to 2.7 branch https://github.com/zephyrproject-rtos/zephyr/blob/v2.7-branch/subsys/bluetooth/mesh/gatt_services.c#L912

There is no need to back port for this, since this issue has been involved by #36871

@cfriedt cfriedt merged commit b29ce22 into zephyrproject-rtos:main Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Bluetooth Mesh area: Bluetooth bug The issue is a bug, or the PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants