On Fri, 2008-09-12 at 23:46 -0300, Luiz Augusto von Dentz wrote: > Both seems good, but please use debug instead of printf. Btw does your > first patch solves the problem? Oops. I meant to remove the printf before I sent the patch :) Have committed both to git://git.infradead.org/~dwmw2/bluez.git without the printf. Before my first patch, look closely at the handling of req->search_uuid in browse_cb(). Note that it's incremented _every_ time we end up back in browse_cb(). Even when we've already finished going through the uuid_list[] array, and we're supposed to be iterating through req->uuids. So what happens is we handle everything in the uuid_list[] until we reach the zero at the end. Then we handle the first item from req->uuids, then we trawl through the memory _after_ uuid_list[], treating it as more uuids to search for until we reach another zero. Then we handle the second item from req->uuids, then... -- David Woodhouse Open Source Technology Centre David.Woodhouse@xxxxxxxxx Intel Corporation -- 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
