Re: [PATCH] Fix possible crash on startup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Bastien,

Thanks for the patch. It has been pushed upstream. However, now that I looked at the code in question there are several things I have to wonder about: 1. Why is the code using sdp_data_get(rec, SDP_SERVER_RECORD_HANDLE) instead of rec->handle? 2. Is having the record really mandatory for allowing a device to connect? I.e. could we somehow make input_probe() work even if the record isn't available?

I think it was either Claudio or Luiz who has been touching this part of bluez code recently (at least git-blame shows Claudio being responsible for the SDP_SERVER_RECORD_HANDLE thing) so maybe they can shed some more light on these issues?

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

[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux