Hi Claudio,

On Fri, Nov 23, 2012, Johan Hedberg wrote:
> On Thu, Nov 22, 2012, Claudio Takahasi wrote:
> > BlueZ supports now external profile registration, see:
> > bluez/doc/profile-api.txt
> > 
> > This patch series has the initial implementation of the HFP external
> > profile in oFono. HandsfreeAgent and HandsfreeGateway interfaces were
> > removed from BlueZ.
> > 
> > The next step: implement SCO connection handling in oFono.
> 
> In general the patches seem quite good, but I get the following when
> connecting HFP, disconnecting and reconnecting (all actions triggered
> from the phone side):
> 
> ofonod[3940]: plugins/hfp_hf.c:profile_new_connection() Profile handler 
> NewConnection
> ofonod[3940]: plugins/hfp_hf.c:profile_new_connection() hfp_data: 0x4e190f0 
> SLC FD: 9 Version: 0x0106 Features: 0x0027
> ==3940== Invalid read of size 8
> ==3940==    at 0x47E565: profile_new_connection (hfp_hf.c:196)
> ==3940==    by 0x40F960: process_message.isra.0 (object.c:197)
> ==3940==    by 0x37F181D874: ??? (in /usr/lib64/libdbus-1.so.3.7.1)
> ==3940==    by 0x37F180FAEF: dbus_connection_dispatch (in 
> /usr/lib64/libdbus-1.so.3.7.1)
> ==3940==    by 0x40DFE7: message_dispatch (mainloop.c:76)
> ==3940==    by 0x37EE0483BA: g_timeout_dispatch (gmain.c:3882)
> ==3940==    by 0x37EE047824: g_main_context_dispatch (gmain.c:2539)
> ==3940==    by 0x37EE047B57: g_main_context_iterate.isra.23 (gmain.c:3146)
> ==3940==    by 0x37EE047F51: g_main_loop_run (gmain.c:3340)
> ==3940==    by 0x40DC41: main (main.c:247)
> ==3940==  Address 0x4e19130 is 64 bytes inside a block of size 96 free'd
> ==3940==    at 0x4A07786: free (vg_replace_malloc.c:446)
> ==3940==    by 0x37EE04D50E: g_free (gmem.c:252)
> ==3940==    by 0x37EE045797: g_source_callback_unref (gmain.c:1288)
> ==3940==    by 0x37EE04528A: g_source_destroy_internal (gmain.c:957)
> ==3940==    by 0x37EE04786F: g_main_context_dispatch (gmain.c:2563)
> ==3940==    by 0x37EE047B57: g_main_context_iterate.isra.23 (gmain.c:3146)
> ==3940==    by 0x37EE047F51: g_main_loop_run (gmain.c:3340)
> ==3940==    by 0x40DC41: main (main.c:247)

I also found this leak:

==3940== 195 (96 direct, 99 indirect) bytes in 1 blocks are definitely lost in 
loss record 134 of 175
==3940==    at 0x4A0881C: malloc (vg_replace_malloc.c:270)
==3940==    by 0x37EE04D55D: g_try_malloc0 (gmem.c:296)
==3940==    by 0x47E2F3: hfp_hf_probe (hfp_hf.c:225)
==3940==    by 0x47B9A4: device_properties_cb (bluetooth.c:307)
==3940==    by 0x37F180C729: ??? (in /usr/lib64/libdbus-1.so.3.7.1)
==3940==    by 0x37F180F7F2: dbus_connection_dispatch (in 
/usr/lib64/libdbus-1.so.3.7.1)
==3940==    by 0x40DFE7: message_dispatch (mainloop.c:76)
==3940==    by 0x37EE0483BA: g_timeout_dispatch (gmain.c:3882)
==3940==    by 0x37EE047824: g_main_context_dispatch (gmain.c:2539)
==3940==    by 0x37EE047B57: g_main_context_iterate.isra.23 (gmain.c:3146)
==3940==    by 0x37EE047F51: g_main_loop_run (gmain.c:3340)
==3940==    by 0x40DC41: main (main.c:247)

Johan
_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono

Reply via email to