Hi Luiz, > On Tue, Dec 9, 2014 at 4:50 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote: > Hi Arman, > > On Tue, Dec 9, 2014 at 2:40 AM, Arman Uguray <armansito@xxxxxxxxxxxx> wrote: >> This patch adds a getter for a bt_att structure's underlying connection >> file descriptor. >> --- >> src/shared/att.c | 8 ++++++++ >> src/shared/att.h | 1 + >> 2 files changed, 9 insertions(+) >> >> diff --git a/src/shared/att.c b/src/shared/att.c >> index ee425d8..9511bb2 100644 >> --- a/src/shared/att.c >> +++ b/src/shared/att.c >> @@ -922,6 +922,14 @@ void bt_att_unref(struct bt_att *att) >> free(att); >> } >> >> +int bt_att_get_fd(struct bt_att *att) >> +{ >> + if (!att) >> + return -EINVAL; >> + >> + return att->fd; >> +} >> + >> bool bt_att_set_close_on_unref(struct bt_att *att, bool do_close) >> { >> if (!att || !att->io) >> diff --git a/src/shared/att.h b/src/shared/att.h >> index 99b5a5b..b946b18 100644 >> --- a/src/shared/att.h >> +++ b/src/shared/att.h >> @@ -33,6 +33,7 @@ struct bt_att *bt_att_new(int fd); >> struct bt_att *bt_att_ref(struct bt_att *att); >> void bt_att_unref(struct bt_att *att); >> >> +int bt_att_get_fd(struct bt_att *att); > > It is not clear why this would be necessary? > I added this to make one call site work where src/device uses it to obtain a socket error in the event of a disconnect. I could probably avoid this by getting the fd out of the GIOChannel which device is still caching anyway. -Arman -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html