Re: svn commit: rev 7886 - in trunk/subversion: include libsvn_subr

dionisos@tigris.org writes:

> +static svn_error_t *
> +do_io_file_wrapper_cleanup (apr_file_t *file, apr_status_t status,
> + const char *op, apr_pool_t *pool)
> +{
> + const char *name;
> + const char *errstr;
> + char errbuf[255];
> + svn_error_t *err;
> +
> + if (! status)
> + return SVN_NO_ERROR;
> +
> + err = file_name_get (&name, file, pool);
> + name = (name && ! err) ? apr_psprintf (pool, "file '%s'", name) : "stream";

Trying to use name if file_name_get returned an error will provoke
undefined behaviour if name has not be assigned a value. Do it like
this

     name = (! err && name) ...

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org