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

fixed a possible infinite loop in todo_get_position()

Browse Source
This commit is contained in:
Frederic Culot 2006-09-02 09:20:35 +00:00
parent 8b6943cc2c
commit cc95f9f8ef
1 changed files with 15 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/todo.c
View File

@ -1,4 +1,4 @@
/* $calcurse: todo.c,v 1.3 2006/08/31 18:47:54 culot Exp $ */ /* $calcurse: todo.c,v 1.4 2006/09/02 09:20:35 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -120,15 +120,23 @@ struct todo_s *todo_get_item(int item_number)
*/ */
int todo_get_position(struct todo_s *i) int todo_get_position(struct todo_s *i)
{ {
struct todo_s *o = todolist; struct todo_s *o;
int n = 1; int n = 1, found = 0;
for (;;) { for (o = todolist; o; o = o->next) {
if (o == i) break; if (o == i) {
o = o->next; found = 1;
break;
}
n++; n++;
} }
return n; if (found) {
return n;
} else {
fputs(_("FATAL ERROR in todo_get_position: todo not found\n"),
stderr);
exit(EXIT_FAILURE);
}
} }
/* Change an item priority by pressing '+' or '-' inside TODO panel. */ /* Change an item priority by pressing '+' or '-' inside TODO panel. */