[Gluster-devel] [PATCH BUG:3020 1/1] Fix duplicate quota/marker symbols.

Emmanuel Dreyfus manu at netbsd.org
Mon Jul 18 14:39:49 UTC 2011


Hi

I am still unable to use your patch, but I spoted two bugs

> +	if (server) {
> +		ret = SSL_accept(priv->ssl_ssl);
> +	}
> +	else {
> +		ret = SSL_connect(priv->ssl_ssl);
> +	}

Here you should  use SSL_get_error() and check for various errors
that require SSL_accept/SSL_connect to be restarted. For instance, 
I could not get openssl s_client to show the server certificate if I do 
not restart on SSL_ERROR_WANT_READ. The connexion just closed immediatly,
and kernel trace show the SSL socket returns EAGAIN on a read(2°.

> +socket_poller (void *ctx)
> +{
> +        rpc_transport_t  *this = ctx;
> +        socket_private_t *priv = this->private;
> +	struct pollfd     pfd[2] = {{0,},};
> +	gf_boolean_t      to_write = _gf_false;
> +	int               ret = 0;
> +	int               orig_gen;

orig_gen may be used uninitialized.
-- 
Emmanuel Dreyfus
manu at netbsd.org




More information about the Gluster-devel mailing list