Hi,
On Tue, Mar 27, 2012, Anderson Lizardo wrote:
> From: Claudio Takahasi <claudio.takahasi@xxxxxxxxxxxxx>
>
> This patch fixes missing reply for DiscoverCharacteristics method on
> Generic Attribute. If link is disconnected reply message is not sent.
> ---
> attrib/client.c | 14 ++++++++++++++
> 1 files changed, 14 insertions(+), 0 deletions(-)
I've applied the first four patches but there was something in this one
that bothered me:
> diff --git a/attrib/client.c b/attrib/client.c
> index ce78ed6..57cd028 100644
> --- a/attrib/client.c
> +++ b/attrib/client.c
> @@ -356,6 +356,20 @@ static void attio_disconnected(gpointer user_data)
> {
> struct gatt_service *gatt = user_data;
>
> + if (gatt->query) {
> + if (gatt->query->msg) {
> + DBusMessage *reply;
> +
> + reply = btd_error_failed(gatt->query->msg,
> + strerror(ECONNRESET));
What's the point of picking a POSIX error and using its glibc string
translation here? Why don't you provide your own string instead which
can be much more understandable for someone who reads the logs, e.g.
"Attribute IO got disconnected" or whatever makes more sense.
Johan
--
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
