diff options
| author |   matthew <matthew@openbsd.org> | 2013-11-25 23:15:58 +0000 |
|---|---|---|
| committer | matthew <matthew@openbsd.org> | 2013-11-25 23:15:58 +0000 |
| commit | 658f91c7e51c6dd70e27ff496e6bb20fd15d1b2f (patch) | |
| tree | 403306cd317d827709d8386889c5c4992f95ce3c | |
| parent | Allow auto install/upgrade in non-netbooted case. (diff) | |
| download | wireguard-openbsd-658f91c7e51c6dd70e27ff496e6bb20fd15d1b2f.tar.xz wireguard-openbsd-658f91c7e51c6dd70e27ff496e6bb20fd15d1b2f.zip | |
Fix infinite loop pointed out by clang/brad.
ok guenther
| -rw-r--r-- | sys/kern/sysv_sem.c | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/sys/kern/sysv_sem.c b/sys/kern/sysv_sem.c index 2789670ddd7..0b6da202ef8 100644 --- a/sys/kern/sysv_sem.c +++ b/sys/kern/sysv_sem.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sysv_sem.c,v 1.44 2012/10/21 19:00:48 beck Exp $ */ +/* $OpenBSD: sysv_sem.c,v 1.45 2013/11/25 23:15:58 matthew Exp $ */ /* $NetBSD: sysv_sem.c,v 1.26 1996/02/09 19:00:25 christos Exp $ */ /* @@ -713,17 +713,16 @@ done: * we applied them. This guarantees that we won't run * out of space as we roll things back out. */ - if (i != 0) { - for (j = i - 1; j >= 0; j--) { - if ((sops[j].sem_flg & SEM_UNDO) == 0) - continue; - adjval = sops[j].sem_op; - if (adjval == 0) - continue; - if (semundo_adjust(p, &suptr, semid, - sops[j].sem_num, adjval) != 0) - panic("semop - can't undo undos"); - } + for (j = i; j > 0;) { + j--; + if ((sops[j].sem_flg & SEM_UNDO) == 0) + continue; + adjval = sops[j].sem_op; + if (adjval == 0) + continue; + if (semundo_adjust(p, &suptr, semid, + sops[j].sem_num, adjval) != 0) + panic("semop - can't undo undos"); } for (j = 0; j < nsops; j++) |