[Coverity CID :219497] Logically dead code in subsys/bluetooth/host/gatt.c

[zephyrbot]

Static code scan issues found in file:

https://github.com/zephyrproject-rtos/zephyr/tree/bd97359a5338b2542d19011b6d6aa1d8d1b9cc3f/subsys/bluetooth/host/gatt.c

Category: Control flow issues
Function: cf_set_value
Component: Bluetooth
CID: 219497

Details:

zephyr/subsys/bluetooth/host/gatt.c

Line 510 in bd97359

cfg->data[i] |= value[i];

Please fix or provide comments in coverity using the link:

https://scan9.coverity.com/reports.htm#v32951/p12996.

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.

[joerchan]

This is logically dead because there is not enough bits yet defined to take this branch. Once more bits are added this code-path will be needed.

@jhedberg You agree that we should close this as intentional?

[github-actions]

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

[carlescufi]

This is logically dead because there is not enough bits yet defined to take this branch. Once more bits are added this code-path will be needed.

@jhedberg You agree that we should close this as intentional?

I've looked at the code now and I agree completely. Will close this.

[carlescufi]

Marked as false positive in Coverity.