scusate secondo voi questo codice è completamente corretto?

<%
Set objConn = Server.CreateObject("ADODB.Connection")
objConn.Open "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=" & Server.MapPath("contatore.mdb")

set objRS=server.createObject("ADODB.Recordset")

'incremento il contatore per sapere quante volte è stato scaricato l'aggiornamento
sql = "UPDATE volte SET scaricato = scaricato +1"
objConn.Execute(sql)
objConn.Close
set objConn=nothing

response.redirect "http://miosito/miofile.exe"

%>