Re: [Valgrind-users] Invalid write of size 4

SourceForge Headquarters 1320 Columbia Street Suite 310 San Diego, CA 92101 +1 (858) 422-6466

hi hi,

bloody stupid me! The line
self->priv->global_names =
   (gchar **    )g_new0(gchar   ,self->priv->global_params);
needs to be
self->priv->global_names =
   (gchar **    )g_new0(gpointer,self->priv->global_params);

What would I do without valgrind and *you*.

Stefan

> hi hi,
> 
> can anyone help me to understand the follwing report from
> valgrind-2.4.0. Earlier in the report I get stuff like:
> Conditional jump or move depends on uninitialised value(s)
> Syscall param writev(vector[...]) points to uninitialised byte(s)
> Those are always in system libs like libc, ld or pthread.
> 
> Then comes this:
> 
> DEBUG (0x1bf43368 - 310459:30:26.946383000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [0/7] "Start
> Freq"
> DEBUG (0x1bf43368 - 310459:30:27.016127000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [0/7] "StartF req"
> DEBUG (0x1bf43368 - 310459:30:27.078957000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [1/7] "EndFr eq"
> ==23367==
> ==23367== Invalid write of size 4
> ==23367==    at 0x1B9126F7: bt_machine_new (machine.c:575)
> ==23367==    by 0x1B927629: bt_source_machine_new (source-machine.c:45)
> ==23367==    by 0x1B9230E9: bt_song_io_native_load_setup_machines
> (song-io-native.c:230)
> ==23367==    by 0x1B9235AC: bt_song_io_native_load_setup
> (song-io-native.c:299)
> ==23367==    by 0x1B9252A5: bt_song_io_native_real_load
> (song-io-native.c:746)
> ==23367==    by 0x1B921EDB: bt_song_io_load (song-io.c:189)
> ==23367==    by 0x80499D2: bt_cmd_application_play
> (cmd-application.c:103)
> ==23367==    by 0x8049505: main (bt-cmd.c:55)
> ==23367==  Address 0x1C10AF84 is 4 bytes inside a block of size 7
> alloc'd
> ==23367==    at 0x1B905E7A: calloc (vg_replace_malloc.c:176)
> ==23367==    by 0x1BD89CBE: g_malloc0
> (in /usr/lib/libglib-2.0.so.0.600.4)
> ==23367==    by 0x1B927629: bt_source_machine_new (source-machine.c:45)
> ==23367==    by 0x1B9230E9: bt_song_io_native_load_setup_machines
> (song-io-native.c:230)
> ==23367==    by 0x1B9235AC: bt_song_io_native_load_setup
> (song-io-native.c:299)
> ==23367==    by 0x1B9252A5: bt_song_io_native_real_load
> (song-io-native.c:746)
> ==23367==    by 0x1B921EDB: bt_song_io_load (song-io.c:189)
> ==23367==    by 0x80499D2: bt_cmd_application_play
> (cmd-application.c:103)
> ==23367==    by 0x8049505: main (bt-cmd.c:55)
> DEBUG (0x1bf43368 - 310459:30:27.201295000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [1/7] "EndFre q"
> DEBUG (0x1bf43368 - 310459:30:27.367592000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [2/7] "Decay "
> DEBUG (0x1bf43368 - 310459:30:27.428693000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [2/7] "Decay"
> DEBUG (0x1bf43368 - 310459:30:27.489094000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [3/7] "FreqD
> ecay"
> DEBUG (0x1bf43368 - 310459:30:27.550750000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [3/7] "FreqDe cay"
> DEBUG (0x1bf43368 - 310459:30:27.665047000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [4/7] "Thump "
> DEBUG (0x1bf43368 - 310459:30:27.727406000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [4/7] "Thump"
> DEBUG (0x1bf43368 - 310459:30:27.790057000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [5/7] "Gain"
> DEBUG (0x1bf43368 - 310459:30:27.852316000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [5/7] "Gain"
> DEBUG (0x1bf43368 - 310459:30:27.915595000)              bt-core(23367)
> machine.c(573):bt_machine_new:     adding global_param [6/7] "Trig"
> DEBUG (0x1bf43368 - 310459:30:28.044170000)              bt-core(23367)
> machine.c(592):bt_machine_new:     added global_param [6/7] "Trig"
> 
> this is the section from machine.c. full source:
> http://cvs.sourceforge.net/viewcvs.py/buzztard/buzztard/src/lib/core/machine.c?view=markup
> line 757 is 'self->priv->global_names[j]=property->name;'
> 
> self->priv->global_names =
> (gchar **    )g_new0(gchar   ,self->priv->global_params);
> self->priv->global_flags =
> (guint *     )g_new0(guint   ,self->priv->global_params);
> for(i=j=0;i<number_of_properties;i++) {
>   property=properties[i];
>   if(property->flags&GST_PARAM_CONTROLLABLE) {
>     GST_DEBUG("    adding global_param [%d/%d] \"%s
> \"",j,self->priv->global_params,property->name);
>     // add global param
>     self->priv->global_names[j]=property->name;
>     self->priv->global_types[j]=property->value_type;
>     ...
>     j++;
>   }
> }
> 
> Many thanks,
> 
> Stefan
> --
> http://www.buzztard.org
> 
> 
> 
> 
> -------------------------------------------------------
> This SF.Net email is sponsored by Yahoo.
> Introducing Yahoo! Search Developer Network - Create apps using Yahoo!
> Search APIs Find out how you can build Yahoo! directly into your own
> Applications - visit http://developer.yahoo.net/?fr=offad-ysdn-ostg-q22005
> _______________________________________________
> Valgrind-users mailing list
> Val...@li...
> https://lists.sourceforge.net/lists/listinfo/valgrind-users