cf_set() returns 0 when no cfg is available

[TTJO]

In gatt.c @ cf_set(), the function returns with success (0) when no cfg is available.

cfg = find_cf_cfg_by_addr(id, &addr); if (!cfg) { cfg = find_cf_cfg(NULL); if (!cfg) { BT_ERR("Unable to restore CF: no cfg left"); return 0; } }

The function should probably return
return -ENOMEM;
when no cfg is available.

The impact on my progress is currently not assessed. Just found this bug as part of another bug-hunt.

[carlescufi]

@TTJO would you mind sending a PR? This is a trivial change and a good first Pull Request.

[TTJO]

Yes... I will do that