>In my application I have several places where ShellExecute is called. Is there a downside (potential problem) with loading this dll multiple times? Should I think of creating a wrapper function to call it only once?

No and yes. The time for DECLARE is negligible compare to time required to launch an application. You should write a wrapper, but for different reasons.