On 03/05/2010 08:19 PM, Sumit Bose wrote:
On Fri, Mar 05, 2010 at 12:00:41PM -0500, Dmitri Pal wrote:
> Sumit Bose wrote:
>> Hi,
>>
>> the following two patches are related to the Kerberos locator plugin.
>>
>> The first patch removes the name resolution from the locator plugin and
>> let it use the IP address found by the resolver code. This way we can be
>> sure to always talk to the same server until the resolver chooses a new
>> server.
>>
>> The second patch tries to make debugging of the locator plugin easier by
>> activating it automatically if the debug level of the Kerberos provider
>> is 5 or higher. Also the timestamp and debug-to-files flags are
>> respected.
>>
>> bye,
>> Sumit
>>
> I had a very quick glance.
> I think you are missing checks after strdup().
>
> ctx->kdc_addr = strdup((char *) buf);
>
>
> I am not familiar with the code so it is hard for me to comment on anything else.
>
oops, new version attached.
Hi, I can't apply the patch, can you please rebase?
Thanks
Martin