Bug 25784 – Leaks seen on HTTP tests

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Alexey Proskuryakov]

There are many leaks seen on HTTP tests. Some of them seem to be in system frameworks, so I'm going to fix some, and file Radar bugs for others.

For those who can see it, one Radar bug is <rdar://problem/6886937>.

[Alexey Proskuryakov]

Created attachment 30320 [details]
proposed fix

This fixes two bugs I found.

[Alexey Proskuryakov]

(In reply to comment #0)
> For those who can see it, one Radar bug is <rdar://problem/6886937>.

Another one: <rdar://problem/6887278>.

[Darin Adler]

Comment on attachment 30320 [details]
proposed fix

> -        NSMutableURLRequest *mutableRequest = [request.nsURLRequest() mutableCopy];
> +        NSMutableURLRequest *mutableRequest = [[request.nsURLRequest() mutableCopy] autorelease];

Clearly this was just introduced by the bug fix for content sniffing. I'd prefer to find a way to fix this with an additional local variable and a normal release rather than autorelease.

r=me as is, but consider making it use release instead of autorelease

[Alexey Proskuryakov]

Committed r43709.

(In reply to comment #3)
> r=me as is, but consider making it use release instead of autorelease

I think that it would be much more interesting - but also more difficult - to avoid autorelease in common code path.