Hola:
Es raro el código que tienes, porque haces un bucle dentro de otro con el mismo conjunto de elementos...
Si el elemento es el que llamas atacar, creo que lo podrías simplificar.
function desactivarActivados()
{
var ataques = document.forms[0].elements['ataque'];
var atacar = document.getElementById('atacar')
.checked;
for(var i = 0; i < ataques.length; i++)
ataques[i].disabled = atacar;
}
}
Así se habiliyan/deshabilitan según esté chequeado el elemento atacar.
Saludos
