Remove ASSERT() in sckClientOpen() to silence Coverity complaint.

Coverity complains that this comparison might have a side effect because the variable is volatile. It's hard to see what that might be but since the assertion is not all that important, just remove it. During testing this sort of error will generally be caught by valgrind.

src/common/io/socket/client.c

@@ -234,7 +234,6 @@ sckClientOpen(THIS_VOID)
             CATCH_ANY()
             {
                 // Close socket
-                ASSERT(openData != NULL);
                 close(openData->fd);
 
                 // Clear socket so the connection can be retried