A mi me parece que esa funcion esta bien. Incluso, se puede quitar el
else 
El error puede estar en esta linea:
document.form1.eliminar.value="";
Quizas tu formulario tenga otro nombre, o el campo no se llame
eliminar.
Dificil dar una solucion, cuando hay pocos datos