Edité tu mensaje para colocar el código entre etiquetas para que la lectura sea más amigable y las personas que lo lean tengan más interés en responder (te sugiero que la próxima vez que postees hagas lo mismo usando las herramientas del editor de texto del foro).
Ahora, yendo al script, es difícil encontrar el error en ese fragmento. A simple vista no veo problemas. Quizá deberías postear el resto de tu código o, mejor aún, usar la consola de errores de firefox (Herramientas->Desarrollador web->Consola de errores) y contarnos qué error te muestra. Las nuevas versiones de Explorer también tienen una buena consola de Errores que podés mirar.