diff options
author | Om Narasimhan <om.turyx@gmail.com> | 2006-10-20 21:44:15 (GMT) |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2006-10-26 01:59:42 (GMT) |
commit | a230a6785dd5af84b8b043a64d8df8adc81f3724 (patch) | |
tree | f882cb5538776f5aa8a88731f7e34a8f2887e3c9 /drivers/pcmcia/at91_cf.c | |
parent | f465ce176fb2f1778a04fc3fcb2b8aa564901419 (diff) | |
download | linux-a230a6785dd5af84b8b043a64d8df8adc81f3724.tar.xz |
[PATCH] pcmcia: au1000_generic fix
The previous code did something like,
if (error) goto out_err;
....
do {
struct au1000_pcmcia_socket *skt = PCMCIA_SOCKET(i);
del_timer_sync(&skt->poll_timer);
pcmcia_unregister_socket(&skt->socket);
out_err:
flush_scheduled_work();
ops->hw_shutdown(skt);
i--;
} while (i > 0)
.....
- On the error path, skt would not contain a valid value for the first
iteration (skt is masked by uninitialized automatic skt)
- Does not do hw_shutdown() for 0th element of PCMCIA_SOCKET
Signed-off-by: Om Narasimhan <om.turyx@gmail.com>
Cc: "Yoichi Yuasa" <yoichi_yuasa@tripeaks.co.jp>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'drivers/pcmcia/at91_cf.c')
0 files changed, 0 insertions, 0 deletions