Hi Flavio,

I tested "edje_player multisense.edje" to reproduce seg fault as mentioned 
below, but couldn't succeed with memcheck as well.
My svn rev is 72374. Please let me know the scenario to reproduce it.

And regarding the point 5, you are right, multisense will not build without 
remix as per current configure file.
But, multisense is not all about only sound, it can have vibration etc. So, if 
we make remix optional for multisense feature,
edje must compile e.g. ENABLE_MULTISENSE=1 and HAVE_REMIX=0.

Regards,
Prince




------------------------------------------------------------
Date: Thu, 14 Jun 2012 17:31:43 -0300
From: Fl?vio Ceolin <flavio.ceo...@profusion.mobi>
Subject: Re: [E-devel] [Patch] [Edje] Added: Pulseaudio remix plug-in
        for edje multisense module
To: Enlightenment developer list
        <enlightenment-devel@lists.sourceforge.net>
Cc: csm...@samsung.com
Message-ID:
        <cakxyohkiyzzfxra7ff8jvsks-xqughyupeegnlidk_q4leo...@mail.gmail.com>
Content-Type: text/plain; charset="iso-8859-1"

Hello Prince,

On Thu, Jun 14, 2012 at 10:14 AM, PRINCE KUMAR DUBEY
<prince.du...@samsung.com> wrote:
>
> Hi,
> ? Please review the attached patch for pulse audio remix pulg-in added as 
> output device for multisense feature with other enhancements, details are 
> given below.
>
> Change Description:
> 1. Multisense support enabled by default, Pulse Audio option added, flac, 
> alsa and ogg/vorbis are made optional for multisense during 
> configuration/compilation.
> 2. Pulse Audio Remix Plug-in added as audio output device.
> 3. Disabling of Multisense module initialization/shutdown in EDC compilation 
> mode.
> 4. Enhancement to _edje_multisense_shutdown to fix memory leak.
> 5. Multisense example enhancement and modification in edje_multisense.c so, 
> edje can compile without availability of remix.
>
> Signed-Off-By: Prince Kr Dubey<prince.du...@samsung.com>
>
> Thank you,
>
> Regards,
> Prince Kr Dubey
>
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>

I' ve found a segfault when running edje_player (multisense example,
multisense.edj), you could check it in the attached file.
It's valid you know that this problem is not happening when running
the edje-multisense's example.
Also, I didn't understand the point 5 (edje can compile without
availability of remix).
As far as I could see there is no way to build multisense without
libremix right ?

Regards,
Flavio Ceolin
-------------- next part --------------
==11191== Memcheck, a memory error detector
==11191== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al.
==11191== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info
==11191== Command: edje_player multisense.edj
==11191== Parent PID: 8099
==11191== 
==11191== Syscall param write(buf) points to uninitialised byte(s)
==11191==    at 0x355100E04D: ??? (in /usr/lib64/libpthread-2.15.so)
==11191==    by 0x4C7251C: _edje_multisense_internal_sound_sample_play 
(edje_multisense.c:344)
==11191==    by 0x4C73EFF: _edje_program_run (edje_program.c:798)
==11191==    by 0x4C74238: _edje_program_run (edje_program.c:595)
==11191==    by 0x4C76007: _edje_emit_handle (edje_program.c:1228)
==11191==    by 0x4C7174E: _edje_message_queue_process 
(edje_message_queue.c:764)
==11191==    by 0x4C718B6: _edje_job (edje_message_queue.c:160)
==11191==    by 0x69CF234: _ecore_job_event_handler (ecore_job.c:115)
==11191==    by 0x69C95DC: _ecore_call_handler_cb (ecore_private.h:319)
==11191==    by 0x69CA57B: _ecore_event_call (ecore_events.c:559)
==11191==    by 0x69D141D: _ecore_main_loop_iterate_internal (ecore_main.c:1814)
==11191==    by 0x69CFCBF: ecore_main_loop_begin (ecore_main.c:931)
==11191==  Address 0x7fefff97c is on thread 1's stack
==11191== 
==11191== Thread 3:
==11191== Invalid write of size 2
==11191==    at 0xE4538E5: pa_player_playbuffer.isra.0 
(pa_simple_snd_player.c:211)
==11191==    by 0xE4539F4: pa_player_process (pa_simple_snd_player.c:264)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E4787: remix_sound_process (remix_sound.c:400)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E3714: remix_layer_process (remix_layer.c:498)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E6B5A: remix_track_twolayer_process (remix_track.c:350)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==  Address 0x77a20f0 is 0 bytes after a block of size 8,192 alloc'd
==11191==    at 0x4A06F18: calloc (vg_replace_malloc.c:566)
==11191==    by 0xE453524: pa_player_reset_device (pa_simple_snd_player.c:81)
==11191==    by 0xE4536B5: pa_player_init (pa_simple_snd_player.c:130)
==11191==    by 0x56DF729: remix_new (remix_base.c:55)
==11191==    by 0x4C727B5: _edje_multisense_init (edje_multisense.c:109)
==11191==    by 0x4C6EF24: edje_init (edje_main.c:73)
==11191==    by 0x4022F3: main (edje_player.c:662)
==11191== 
==11191== Invalid read of size 4
==11191==    at 0xE4538ED: pa_player_playbuffer.isra.0 
(pa_simple_snd_player.c:208)
==11191==    by 0xE4539F4: pa_player_process (pa_simple_snd_player.c:264)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E4787: remix_sound_process (remix_sound.c:400)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E3714: remix_layer_process (remix_layer.c:498)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E6B5A: remix_track_twolayer_process (remix_track.c:350)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==  Address 0x7778f78 is 0 bytes after a block of size 16,424 alloc'd
==11191==    at 0x4A06F18: calloc (vg_replace_malloc.c:566)
==11191==    by 0xE453638: pa_player_init (pa_simple_snd_player.c:105)
==11191==    by 0x56DF729: remix_new (remix_base.c:55)
==11191==    by 0x4C727B5: _edje_multisense_init (edje_multisense.c:109)
==11191==    by 0x4C6EF24: edje_init (edje_main.c:73)
==11191==    by 0x4022F3: main (edje_player.c:662)
==11191== 
==11191== Invalid read of size 4
==11191==    at 0xE4538D0: pa_player_playbuffer.isra.0 
(pa_simple_snd_player.c:210)
==11191==    by 0xE4539F4: pa_player_process (pa_simple_snd_player.c:264)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E4787: remix_sound_process (remix_sound.c:400)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E3714: remix_layer_process (remix_layer.c:498)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E6B5A: remix_track_twolayer_process (remix_track.c:350)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==  Address 0x7778f7c is 4 bytes after a block of size 16,424 alloc'd
==11191==    at 0x4A06F18: calloc (vg_replace_malloc.c:566)
==11191==    by 0xE453638: pa_player_init (pa_simple_snd_player.c:105)
==11191==    by 0x56DF729: remix_new (remix_base.c:55)
==11191==    by 0x4C727B5: _edje_multisense_init (edje_multisense.c:109)
==11191==    by 0x4C6EF24: edje_init (edje_main.c:73)
==11191==    by 0x4022F3: main (edje_player.c:662)
==11191== 
==11191== Invalid read of size 8
==11191==    at 0x56E3760: remix_layer_process (remix_layer.c:568)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E6B5A: remix_track_twolayer_process (remix_track.c:350)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E160E: remix_deck_onetrack_process (remix_deck.c:318)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x4C722DA: _player_job (edje_multisense.c:302)
==11191==    by 0x69D4AE3: _ecore_short_job (ecore_thread.c:425)
==11191==    by 0x69D4DDD: _ecore_thread_worker (ecore_thread.c:525)
==11191==  Address 0x70 is not stack'd, malloc'd or (recently) free'd
==11191== 
==11191== 
==11191== Process terminating with default action of signal 11 (SIGSEGV)
==11191==  Access not within mapped region at address 0x70
==11191==    at 0x56E3760: remix_layer_process (remix_layer.c:568)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E6B5A: remix_track_twolayer_process (remix_track.c:350)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x56E160E: remix_deck_onetrack_process (remix_deck.c:318)
==11191==    by 0x56DFA9C: remix_process_fast (remix_base.c:434)
==11191==    by 0x56DFB29: remix_process (remix_base.c:450)
==11191==    by 0x4C722DA: _player_job (edje_multisense.c:302)
==11191==    by 0x69D4AE3: _ecore_short_job (ecore_thread.c:425)
==11191==    by 0x69D4DDD: _ecore_thread_worker (ecore_thread.c:525)
==11191==  If you believe this happened as a result of a stack
==11191==  overflow in your program's main thread (unlikely but
==11191==  possible), you can try to increase the size of the
==11191==  main thread stack using the --main-stacksize= flag.
==11191==  The main thread stack size used in this run was 8388608.
==11191== 
==11191== HEAP SUMMARY:
==11191==     in use at exit: 1,727,013 bytes in 3,975 blocks
==11191==   total heap usage: 9,709 allocs, 5,734 frees, 5,088,038 bytes 
allocated
==11191== 
==11191== LEAK SUMMARY:
==11191==    definitely lost: 9,152 bytes in 17 blocks
==11191==    indirectly lost: 20,960 bytes in 149 blocks
==11191==      possibly lost: 576 bytes in 2 blocks
==11191==    still reachable: 1,696,325 bytes in 3,807 blocks
==11191==         suppressed: 0 bytes in 0 blocks
==11191== Rerun with --leak-check=full to see details of leaked memory
==11191== 
==11191== For counts of detected and suppressed errors, rerun with: -v
==11191== Use --track-origins=yes to see where uninitialised values come from
==11191== ERROR SUMMARY: 3046 errors from 5 contexts (suppressed: 3 from 3)

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to