Hi Luiz, I apology for the mistake of commit message. I had misread the source code. I sent out v2 (http://marc.info/?l=linux-bluetooth&m=150182142229855&w=2). Regards, Eramoto On 08/04/2017 10:20 AM, ERAMOTO Masaya wrote: > Hi Luiz, > > On 08/03/2017 06:30 PM, Luiz Augusto von Dentz wrote: >> Hi Aram, >> >> On Thu, Aug 3, 2017 at 7:10 AM, ERAMOTO Masaya >> <eramoto.masaya@xxxxxxxxxxxxxx> wrote: >>> Fix the following problem that occurs in the case of D-Bus fails to >>> register media player path with org.bluez.MediaPlayer1. >> >> Has this actually happened? Im asking because if that does happen it >> probably because the already exists and we might want to return it >> instead, though the fix seems good and I might apply it anyway. > > I do not know what actually happened. I got this message of valgrind and > forgot to get log messages of bluetoothd when the memory leak happened. > This commit message describe one of cause that I guess from implement of > create_ct_player(). > > Is it better to remove this guessed cause from this patch to prevent > misunderstanding? -- 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
