Obrigatório não é, mas é comum fechar assim que não precisa mais usar naquele momento. A corrupção se dá por outros motivos, até porque se estivesse aberto nem poderia acessar de novo. Inclusive não precisa fechar explicitamente porque já está fazendo implicitamente quando usa o [`using`][1]. O `OleDbCommand` deveria estar em um `using` também. O `File.Exists(sFileXLSX)` causa uma [condição de corrida][2]. Pode não ter uma relação direta com o problema, mas pode estar interferindo nisto, ou senão ainda pode trazer problemas futuros. Eu tiraria a captura desta exceção, está fazendo nada. Na verdade está sim, está trazendo problema para depuração por modificar o *stack trace*. [1]: https://pt.stackoverflow.com/q/422273/101 [2]: https://pt.stackoverflow.com/q/159342/101