Still 1204 and 1024 ?<br>
<br>Indeed, I'm sorry about that.<br><br>That strlen does not seem right at all, it should be buffer maximum so<br>
1024. I would use snprintf in all those key/val printf instead of just<br>
with strings.<br><br>Same thing.<br><br><div class="gmail_quote">On Thu, Mar 8, 2012 at 8:11 AM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org">ileoo@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

On Wed, Mar 07, 2012 at 11:05:58PM +0100, Samuel Pitoiset wrote:<br>
<br>
Hi,<br>
<div class="im"><br>
> ---<br>
>  modules/misc/audioscrobbler.c |  268 +++++++++++++++++++++++++++++++++++++++++<br>
>  1 file changed, 268 insertions(+)<br>
<br>
</div><div class="im">> +++ b/modules/misc/audioscrobbler.c<br>
> @@ -1181,6 +1181,274 @@ static void HandleInterval(mtime_t *next, unsigned int *i_interval)<br>
<br>
</div><div class="im">> +    for (i_song = 0 ; i_song < p_sys->i_songs ; i_song++)<br>
> +    {<br>
> +        audioscrobbler_song_t *p_song = &p_sys->p_queue[i_song];<br>
> +        char psz_key[1204], psz_val[1024];<br>
<br>
</div>Still 1204 and 1024 ?<br>
<div class="im"><br>
> +<br>
> +        /* artist */<br>
> +        sprintf(psz_key, "artist[%d]", i_song);<br>
> +        snprintf(psz_val, strlen(p_song->psz_a) + 1, "%s", p_song->psz_a);<br>
<br>
</div>That strlen does not seem right at all, it should be buffer maximum so<br>
1024. I would use snprintf in all those key/val printf instead of just<br>
with strings.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Ilkka Ollakka<br>
"By the time they had diminished from 50 to 8, the other dwarves began<br>
to suspect 'Hungry' ..."<br>
                -- Gary Larson, "The Far Side"<br>
</font></span><br>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div>Best regards,</div>Samuel Pitoiset.<br><br>