Code review: 7200720: crash in net.dll during NTLM authentication

[Dmitry Samersoff]

Michael,

NTLMAuthentication.java:

1. Copyright year should be either fixed or not changed
2. I think it's better to remove bug id from comments and replace
   comment like
      // MMM 7200720 ??
   to something better.

NTLMAuthSequence.c

1. Copyright ...


Otherwise looks good for me.

-Dmitry


On 2012-11-28 17:47, Michael McMahon wrote:
> On 23/11/12 16:16, Alan Bateman wrote:
>> On 23/11/2012 15:59, Michael McMahon wrote:
>>> Could I get the following change reviewed please?
>>> There was a crash caused by accessing freed memory when
>>> authentication was repeated in the same HttpURLConnection instance/
>>>
>>> http://cr.openjdk.java.net/~michaelm/7200720/webrev.1/
>> I haven't reviewed the changes but I notice this changes the build
>> slightly so I just wonder if you've checked the new build.
>>
>> -Alan
> 
> just checked the new build on Linux and it's fine.
> There are a couple of other fixes that will follow this one
> related to this problem. But, I'd like to go ahead and push this one first
> if possible
> 
> - Michael


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* Give Rabbit time, and he'll always get the answer