diff options
| author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2007-01-27 08:00:01 (GMT) |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-01-30 14:36:01 (GMT) |
| commit | 0c0b3ae68ec93b1db5c637d294647d1cca0df763 (patch) | |
| tree | d865c12ab25c8eb235342c46bfd3ad60c1bfb384 /drivers/net | |
| parent | 5fc7d61aee1a7f7d3448f8fbccaa93371ebeecb0 (diff) | |
| download | linux-fsl-qoriq-0c0b3ae68ec93b1db5c637d294647d1cca0df763.tar.xz | |
net: ifb error path loop fix
On error we should start freeing resources at [i-1] not [i-2].
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/ifb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ifb.c b/drivers/net/ifb.c index ca2b21f..c4ca7c9 100644 --- a/drivers/net/ifb.c +++ b/drivers/net/ifb.c @@ -271,8 +271,7 @@ static int __init ifb_init_module(void) for (i = 0; i < numifbs && !err; i++) err = ifb_init_one(i); if (err) { - i--; - while (--i >= 0) + while (i--) ifb_free_one(i); } |