Philip Martin <philip@codematters.co.uk> writes:
> cmpilato@tigris.org writes:
>
>> Author: cmpilato
>> Date: Fri Oct 10 19:55:29 2003
>> New Revision: 7379
>
>> Modified: trunk/subversion/libsvn_ra_dav/props.c
>> ==============================================================================
>> --- trunk/subversion/libsvn_ra_dav/props.c (original)
>> +++ trunk/subversion/libsvn_ra_dav/props.c Fri Oct 10 19:55:29 2003
>
>> if (svn_path_is_empty (path_s->data))
>> {
>> - /* entire URL was bogus; not a single part of it exists in
>> - the repository! */
>> - err = svn_error_createf(SVN_ERR_RA_ILLEGAL_URL, NULL,
>> - "No part of path '%s' was found in "
>> - "repository HEAD.", parsed_url.path);
>> - ne_uri_free(&parsed_url);
>> - return err;
>> + err = svn_error_createf (SVN_ERR_RA_ILLEGAL_URL, NULL,
>> + "No part of path '%s' was found in "
>> + "repository HEAD.", parsed_path);
>> + goto error;
>
> ../svn/subversion/libsvn_ra_dav/props.c:638: warning: `parsed_path' might be used uninitialized in this function
>
> parsed_path is never assigned a value.
If I were to fix this I'd do it by putting parsed_url.path back. I
haven't done this yet because a) I don't know why it was changed in
the first place, and b) I can't work out how to trigger the code in
question.
--
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 16 21:32:43 2003