Hi Alexandros, On Mon, Jan 21, 2013 at 3:44 PM, Alexandros Antonopoulos <alexandros.antonopoulos@xxxxxxxxxxxxxxxx> wrote: > If the user calls Device1.DisconnectProfile with an invalid profile > uuid disconnect_profile still tries to parse the uuid resulting in > a SIGSEGV > > --- > src/device.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/device.c b/src/device.c > index 3675616..1771c0f 100644 > --- a/src/device.c > +++ b/src/device.c > @@ -1270,6 +1270,9 @@ static DBusMessage *disconnect_profile(DBusConnection *conn, DBusMessage *msg, > > uuid = bt_name2string(pattern); > > + if (uuid == NULL) > + return btd_error_invalid_args(msg); > + > p = find_connectable_profile(dev, uuid); > g_free(uuid); > > -- > 1.8.1 All four patches are now upstream, Ive changed the commit message for this one to include the prefix and also please next time state it is a fix and provide the backtrace so we know the severity of the problem. btw, thanks for the patches. -- Luiz Augusto von Dentz -- 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
