# HG changeset patch # User reimar # Date 1267350067 0 # Node ID a375d947381bc9128bae1ddf42e8f6ed81c56556 # Parent b9ec8956164f5f75ec737dddc7fca997b900aea7 Use negative return values for errors in pls_read_entry to make it match with the checks later on. diff -r b9ec8956164f -r a375d947381b playtreeparser.c --- a/playtreeparser.c Sun Feb 28 09:39:30 2010 +0000 +++ b/playtreeparser.c Sun Feb 28 09:41:07 2010 +0000 @@ -252,14 +252,14 @@ v = pls_entry_get_value(line); if(!v) { mp_msg(MSGT_PLAYTREE,MSGL_ERR,"No value in entry %s\n",line); - return 0; + return -1; } num = atoi(line); if(num <= 0 || num > limit) { if (max_entry >= limit) { mp_msg(MSGT_PLAYTREE, MSGL_WARN, "Too many index entries\n"); - return 0; + return -1; } num = max_entry+1; mp_msg(MSGT_PLAYTREE,MSGL_WARN,"No or invalid entry index in entry %s\nAssuming %d\n",line,num); @@ -267,7 +267,7 @@ if(num > max_entry) { e = realloc(e, num * sizeof(pls_entry_t)); if (!e) - return 0; + return -1; memset(&e[max_entry],0,(num-max_entry)*sizeof(pls_entry_t)); max_entry = num; }