Ciao carmi,

le checkbox vanno da 1 a num_colori, nel ciclo for tu vai da 0 a num_colori-1 e poi mancava una parentesi graffa chiusa

codice:
function closePop(num_colori) { 
  var f = document.form_colori; 

  var rv_colore = "" 

  cont=1; 

  for (var i=1;i<=num_colori;i++) { 

    rv_colore = eval("f.colore"+i+".checked") 

    if (rv_colore == true) { 
      if (cont>1) { 
        alert("Attenzione, non puoi selezionare più di 2 colori!"); 
        return false; 
      } else { 


        cont++; 
      } 
    } 

  } 
}
poi eval è deprecato... al posto di
codice:
rv_colore = eval("f.colore"+i+".checked")
ti consiglio la sintassi
codice:
rv_colore = f.elements['colore'+i].checked