Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: Suspicious code in playlist.c?

Re: Suspicious code in playlist.c?

From: Hardeep Sidhu <hardeeps_at_pobox.com>
Date: Mon, 03 Apr 2006 07:26:41 -0700

postmaster_at_diffenbach.org wrote:
> 1. This code in playlist.c look suspicious: playlist->amount is only
> tested against playlist->max_playlist_size AFTER playlist_>amount is
> used as an array index. (Lines 510, 513, 515.)
>
> 2. And, if amount >= max_playlist_size, the function returns without
> calling queue_post. Shouldn't we be able to use playlists up to and
> including max_playlist_size? Instead we can only use playlists up to
> max_playlist_size - 1.
>
> 3. Shouldn't we either be calling queue_post or reverting
> playlist_info's state if we have to end because max_playlist_size has
> been reached?

Thanks for investigating. I've committed fixes for all these problems.

-Hardeep
Received on 2006-04-03

Page template was last modified "Mon Jun 14 16:30:06 2021" The Rockbox Crew -- Privacy Policy