fyb/calcurse-edge
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix bad use of unsigned integers.

Browse Source 
Unsigned values should never be compared to values less than zero.
Detected with "find_unsigned.cocci" spatch from
http://coccinelle.lip6.fr/impact_linux.php.

Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
This commit is contained in:
Lukas Fleischer 2011-04-05 09:37:20 +02:00
parent 6fb0f6f4c6
commit a5a9ee60d6
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/custom.c
View File

@ -270,9 +270,10 @@ custom_load_conf (struct conf *conf, int background)
var = 0;
break;
case CUSTOM_CONF_PERIODICSAVE:
conf->periodic_save = atoi (e_conf);
if (conf->periodic_save < 0)
if (atoi (e_conf) < 0)
conf->periodic_save = 0;
else
conf->periodic_save = atoi (e_conf);
var = 0;
break;
case CUSTOM_CONF_CONFIRMQUIT:

2
src/mem.c
View File

@ -95,8 +95,6 @@ stats_del_blk (unsigned id)
{
struct mem_blk *o, **i;
EXIT_IF (id < 0, _("Incorrect block id"));
i = &mstats.blk;
for (o = mstats.blk; o; o = o->next)
{