Resolves: #240512

Summary: schema replication op error logs wrong error
Description:
As suggested by Ulf in his original comment, put break in the case
CONN_OPERATION_FAILED and set the macro to return_value for the readability.

ldap/servers/plugins/replication/repl5_connection.c

@@ -1458,6 +1458,8 @@ conn_push_schema(Repl_Connection *conn, CSN **remotecsn)
 							"%s: Schema replication update failed: %s\n",
 							agmt_get_long_name(conn->agmt),
 							ldaperr == -1 ? "Unknown Error" : ldap_err2string(ldaperr));
+						return_value = CONN_OPERATION_FAILED;
+						break;
 						}
 					case CONN_NOT_CONNECTED:
 						return_value = CONN_NOT_CONNECTED;