On Tue, 2011-06-07 at 11:49 +0200, Daniele Forsi wrote: > 2011/6/7 Bastien Nocera: > > > diff --git a/src/adapter.c b/src/adapter.c > > > @@ -938,38 +884,29 @@ void adapter_update_local_name(struct btd_adapter *adapter, const char *name) > > > + int err = adapter_ops->set_name(adapter->dev_id, name); > > + if (err < 0) > > + return -err; > > this should be: > return err; > else the error isn't detected in the calling code: Good catch, fixed now. > > + err = adapter_update_local_name (adapter, name); > > + if (err < 0) > > + return btd_error_failed(msg, strerror(-err)); > -- 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