On Mon, 11 Dec 2006, dionisos@tigris.org wrote:
...
> ra_dav-refactoring: Fix the assert() [the Neon resource may be NULL, not ours].
...
> --- branches/ra_dav-refactoring/subversion/libsvn_ra_dav/util.c (original)
> +++ branches/ra_dav-refactoring/subversion/libsvn_ra_dav/util.c Mon Dec 11 12:45:43 2006
> @@ -244,7 +244,8 @@
> req->method = apr_pstrdup(req->pool, method);
> req->url = apr_pstrdup(req->pool, url);
>
> - assert(req != NULL);
> + /* Neon resources may be NULL on out-of-memory */
> + assert(req->req != NULL);
> apr_pool_cleanup_register(reqpool, req,
> dav_request_cleanup,
> apr_pool_cleanup_null);
This type of problem would be less likely to come up if the Neon
request was named to differentiate it from our libsvn_ra_dav request
(e.g. req->ne_req). We currently seem to use req->req in quite a few
places...
- application/pgp-signature attachment: stored
Received on Mon Dec 11 22:05:21 2006