[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix issue #2242.

From: Senthil Kumaran S <senthil_at_collab.net>
Date: Thu, 26 Jun 2008 11:13:57 +0530

Hi Daniel,

Daniel Shahaf wrote:
> Thanks for your patience with this patch. I'll look at it now.

This is really a good news! I am really happy :)

>> apr_pool_t *pool)
>> {
>> svn_string_t *str;
>> - str = apr_hash_get(creds, SVN_AUTH__AUTHFILE_PASSWORD_KEY,
>> + str = apr_hash_get(creds, SVN_AUTH__AUTHFILE_USERNAME_KEY,
>> APR_HASH_KEY_STRING);
>> - if (str && str->data)
>> + if (strcmp(str->data, username) == 0)
> ^
>
> Can STR be NULL here? I prefer
>
> str = apr_hash_get(creds, SVN_AUTH__AUTHFILE_USERNAME_KEY,
> APR_HASH_KEY_STRING);
> if (str && strcmp(str->data, username) == 0)

Yes a check for STR is nice here. I am +1 for this.

>> {
>> - *password = str->data;
>> - return TRUE;
>> + str = apr_hash_get(creds, SVN_AUTH__AUTHFILE_PASSWORD_KEY,
>> + APR_HASH_KEY_STRING);
>> + if (str && str->data)
>> + {
>> + *password = str->data;
>> + return TRUE;
>> + }
>> }
>> return FALSE;
>> }
>>
>
> Can you write a regression test?

With pleasure. I shall do it asap.

Once again thanks for your time in reviewing this patch.

-- 
Senthil Kumaran S
http://www.stylesen.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-26 07:44:57 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.