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`][2] deveria estar em um `using` também.

O [`File.Exists(sFileXLSX)`][3] causa uma [condição de corrida][4]. 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*.

[Coloquei no **GitHub** para referência futura][5].


  [1]: https://pt.stackoverflow.com/q/422273/101
  [2]: https://learn.microsoft.com/en-us/dotnet/api/system.data.oledb.oledbcommand?view=dotnet-plat-ext-8.0&WT.mc_id=DOP-MVP-5002397
  [3]: https://learn.microsoft.com/en-us/dotnet/api/system.io.file.exists?view=net-8.0&WT.mc_id=DOP-MVP-5002397
  [4]: https://pt.stackoverflow.com/q/159342/101
  [5]: https://github.com/maniero/SOpt/blob/master/CSharp/Conceptual.md