ti propongo un'alternativa, in un'unica funzione e senza variabili globali:

codice:
function doall(f, s){
 var e=f.elements;
 for(var i=0;i<e.length;i++) if(e[i].type=="checkbox"&&e[i].name=="cb") e[i].checked=s;
}
da usare cosi':

codice:
onclick="doall(this.form, this.checked);"
sicuro che abbia senso usare checkbox con lo stesso nome?

ciao