Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a broken conditional that confused coverity. It cannot happen, anyway. 

CID 178.
This commit is contained in:
cube 2006-03-29 09:16:36 +00:00
parent 29b4acda94
commit c002406624
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
usr.bin/config/sem.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: sem.c,v 1.21 2006/03/19 23:36:10 cube Exp $ */ /* $NetBSD: sem.c,v 1.22 2006/03/29 09:16:36 cube Exp $ */
/* /*
* Copyright (c) 1992, 1993 * Copyright (c) 1992, 1993
@ -1199,13 +1199,14 @@ remove_devi(struct devi *i)
struct nvlist *nv, *onv; struct nvlist *nv, *onv;
/* Double-linked nvlist anyone? */ /* Double-linked nvlist anyone? */
for (nv = p->p_devs; nv->nv_ptr != NULL; nv = nv->nv_next) { for (nv = p->p_devs; nv->nv_next != NULL; nv = nv->nv_next) {
if (nv->nv_next && nv->nv_next->nv_ptr == i) { if (nv->nv_next && nv->nv_next->nv_ptr == i) {
onv = nv->nv_next; onv = nv->nv_next;
nv->nv_next = onv->nv_next; nv->nv_next = onv->nv_next;
nvfree(onv); nvfree(onv);
break; break;
} if (nv->nv_ptr == i) { }
if (nv->nv_ptr == i) {
/* nv is p->p_devs in that case */ /* nv is p->p_devs in that case */
p->p_devs = nv->nv_next; p->p_devs = nv->nv_next;
nvfree(nv); nvfree(nv);