diff options
| author | Mike Larkin <mlarkin@cvs.openbsd.org> | 2013-11-13 05:26:24 +0000 |
|---|---|---|
| committer | Mike Larkin <mlarkin@cvs.openbsd.org> | 2013-11-13 05:26:24 +0000 |
| commit | 695b6efeb842435ade3da185213bd05c967868c5 (patch) | |
| tree | 42f63c1b693aa9cf5309846916b8df15e566e3a0 /sys/dev | |
| parent | 7f71227310acc41883ac227ec15886adc4910f27 (diff) | |
Use of uninitialized variable. There are obvious locking problems also
present in this function, but I've been advised to walk away, and it
have been this way forever (and this code is not even enabled in GENERIC
by default anyway)
original bug found by Maxime Villard, thanks.
Diffstat (limited to 'sys/dev')
| -rw-r--r-- | sys/dev/ic/aac.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/ic/aac.c b/sys/dev/ic/aac.c index bf1938d1b4f..c7c436f934c 100644 --- a/sys/dev/ic/aac.c +++ b/sys/dev/ic/aac.c @@ -1,4 +1,4 @@ -/* $OpenBSD: aac.c,v 1.55 2013/05/30 16:15:02 deraadt Exp $ */ +/* $OpenBSD: aac.c,v 1.56 2013/11/13 05:26:23 mlarkin Exp $ */ /*- * Copyright (c) 2000 Michael Smith @@ -1186,7 +1186,7 @@ aac_alloc_commands(struct aac_softc *sc) { struct aac_command *cm; struct aac_fibmap *fm; - int i, error; + int i, error = 0; if (sc->total_fibs + AAC_FIB_COUNT > sc->aac_max_fibs) return (ENOMEM); |