-----BEGIN PGP SIGNED MESSAGE-----
Julian Foad wrote:
> But that's not right. Like I said before, the variable holds different
> kinds of data at different times. (If we this variable was intended to
> always hold a URL, we would want the variable's name to indicate so.)
> It is wrong to call svn_uri_canonicalize() on a string that is not known
> to be a URL. Depending on what the string is, that might change it in
> some undesired way.
Thought cdata->data only holds a URL(after chasing their values in gdb).
Thanks for the clarification. So, canonicalization should be done in the
two places where required rather than doing it initially? If so, will
send the updated patch.
>> Though there's one more place:
>> case ELEM_status:
>> /* Parse the <status> tag's CDATA for a status code. */
>> if (ne_parse_statusline(cdata, &status))
>> return svn_error_create(SVN_ERR_XML_MALFORMED, NULL, NULL);
>> that does not need the canonicalized value, thought its better to do the
>> canonicalization in just one place.
> No. (And there are several other places where the variable is used in
> that function, where the value it holds is something other than a URL.)
Yup, saw them after sending the mail :( Thank you Julian.
Thanks & Regards,
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----END PGP SIGNATURE-----
Received on 2010-02-05 06:25:23 CET