ASI内存泄漏处理----转

原文地址

发现问题

iOS7发布后,我们对产品进行了iOS7的适配。适配完成之后的某天,我使用Leaks对产品的新版本进行内存泄漏检测时发现ASIHTTPRequest存在内存泄漏问题,当时使用的设备是iTouch5,系统为iOS7.0.2。

Leaks检测结果

Leaks

(ps:使用的是ASIHTTPRequest iPhoneSample的检测图,结果是一样的)

发现之初,我以为是某处ASIHTTPRequest使用不当导致的泄漏,于是把leaks中的堆栈全部都检查了一边,但没有发现任何产品工程中的代码(其中一处泄漏的堆栈如图)。

Leaks中StackTrace结果

StackTrace

由于在iOS7发布之前的所有版本中并未看到类似的内存泄漏,所以我就开始怀疑是ASIHTTPRequest在iOS7才产生的。于是我在iOS5和iOS6的设备上进行了Leak Profile,结果没有发现任何泄漏。

对于这样的结果我仍然不是很确信,因为项目的需要我们对ASIHTTPRequest进行了一定的定制,修改了其中一部分代码。为了确定问题确实是出在ASIHTTPRequest上,我去github上翻出了ASIHTTPRequest的repo,pull了最新的代码,用Leaks在iOS7系统上进行了profile。在profile过程中我对iPhone Sample中的每个Tab以此进行了测试,结果在 Synchronous 和 Queue 上并没有发现内存泄漏,在 Upload 上发现了和之前一样泄漏。随后在iOS5和iOS6上也进行了一样的测试,结果依然是没有任何泄漏。

自此确定了这是ASIHTTPRequest在iOS7下特有的内存泄漏,并且只会出现在有POST body的情况下。

寻找解答

发现问题之后,我仔细查看了Leaks Profile的结果,发现泄漏集中在 ASIInputStream 上,应该是在使用的过程中 release 方法在某种情况下没有被调用到。于是我重写了ASIIputStream的release方法并在其中断点,分别在iOS7和iOS6进行调试后发现iOS7比iOS6少了一次Release的调用,堆栈如图所示。

缺少的Release的断点Stack Trace结果

Release

从堆栈来看似乎是 CoreReadStreamFromCFReadStream 这个类的析构函数在iOS7下没有被调用到。当时就感觉没救了,这是私有类,想要强行触发析构似乎是不可能的,能做的就是尽量减少其中的泄漏。于是我们做了如下修改:

- (void)close

{

[stream close];

[stream release];

stream = nil;

}

修改了 ASIInputStream 的 close 方法,在close完成后把其中的 NSInputStream 对象release掉,以减少内存泄漏,同时保证ASIHTTPRequest不被重复使用(因为其中的NSInputStream已经被release了无法再使用)。

这样一来泄漏有了一定的减少,如图。

修改后的Leaks检测结果

Leakss

但这样并不能真正解决问题,泄漏依然存在,但我一时也想不到很好的办法,于是只能给repo发issue期望能够得到原作者的回复(虽然我知道希望不大,这哥们很久没管这事了- -)。 这是我发的issue: https://github.com/pokeb/asi-http-request/issues/378

解决问题

发issue大约一周后的某一天收到github的邮件,说有人回复我的issue了,进去一看有一个好心人这样解答道:

@mjohnson12

The leak is because ASIInputStream is being cast to a CFReadStreamRef but ASIInputStream does not derive from NSInputStream it just wraps it.

My Solution is to get rid of ASIInputStream and create a NSInputStream instead in the ASIHTTPRequest startRequest: method.

It breaks using the metrics that ASIInputStream records but I wasn't using them.

I'm using a fairly old version of ASIHTTPRequest v.1.6.2 so your milage may vary.

于是我按照他的做法,把ASIHTTPRequest里的 ASIInputStream 全部替换成了 NSInputStream ,再用Leaks Profile的时候泄漏果真消失了。也正如这位仁兄所说的, ASIInputStream 只是把自己伪装成一个 NSInputStream ,并且实现了一些 NSInputStream 的接口,实际都是由其中包含的 NSInputStream 实例完成的。 ASIInputStream 这个类的功能主要是用来做流量限制,如果不需要这个功能的话,直接把 ASIInputStream 替换成 NSInputStream 即可解决问题。

那么如果我把 ASIInputStream 继承自 NSInputStream 的话是不是就能既保留流量限制功能又解决泄漏问题了呢?于是我开始尝试继承 NSInputStream ,其中碰到了一些困难。NSInputStream的init方法都是写在一个Category里的,无法被继承- -!。

@interface NSInputStream (NSInputStreamExtensions)

- (id)initWithData:(NSData *)data;

- (id)initWithFileAtPath:(NSString *)path;

- (id)initWithURL:(NSURL *)url NS_AVAILABLE(10_6, 4_0);

+ (id)inputStreamWithData:(NSData *)data;

+ (id)inputStreamWithFileAtPath:(NSString *)path;

+ (id)inputStreamWithURL:(NSURL *)url NS_AVAILABLE(10_6, 4_0);

@end

经过一番google我在Git上发现了一个repo: https://github.com/bjhomer/HSCountingInputStream

这个repo中实现了对于 NSInputStream 的继承。仔细阅读完成后发现其实所谓的继承也只不过时在类的@interface中声明了一下继承自 NSInputStream 而已,实际的工作还是由类中的一个 NSInputStream 实例完成的,但相比于 ASIInputStream 这个repo里的实现多了几个方法:

//私有的CFRunLoopRef schedule、unschedule方法

- (void)_scheduleInCFRunLoop:(CFRunLoopRef)aRunLoop forMode:(CFStringRef)aMode;

- (BOOL)_setCFClientFlags:(CFOptionFlags)inFlags

callback:(CFReadStreamClientCallBack)inCallback

context:(CFStreamClientContext *)inContext;

- (void)_unscheduleFromCFRunLoop:(CFRunLoopRef)aRunLoop forMode:(CFStringRef)aMode;

//NSInputStream的代理方法

- (void)stream:(NSStream *)aStream handleEvent:(NSStreamEvent)eventCode;

这些方法的作用大家可以参考链接: http://blog.octiplex.com/2011/06/how-to-implement-a-corefoundation-toll-free-bridged-nsinputstream-subclass/ 。

简单的说就是 NSInputStream 需要同时支持 NSRunLoop 和 CFRunLoopRef 的schedule和unschedule方法,那几个私有方法就是负责 CFRunLoopRef 的schedule,NSInputStream的代理方法则是负责事件的传递。这使我联想到了之前提到的没有被调用的 release 方法,iOS6下它的堆栈中正好有 CFRunLoopRef 的一些方法。莫非这就是 ASIInputStream 内存泄漏的原因所在?

我当时的猜测是,iOS7下的内存泄漏是 ASIInputStream 伪装的不够像而导致的,之所以这么认为因为虽然有泄漏但 ASIInputStream 的功能依然存在,HTTP请求并未因此失效,这说明 ASIInputStream 还是被bridge成了 CFReadStream ,但只是因为少了和 NSInputStream 一样的unschedule方法导致其没有被正常unschedule。如果我把 ASIInputStream 的unschedule方法补上是否就可以解决问题?

我把上述的几个方法在 ASIInputStream 中实现了以后再进行Leaks Profile,内存泄漏果然如预期的那样消失了!在 ASIInputStream 的 release 方法中打断点后发现也能够正常调用了。问题到这里算是解决了。接下来要解决这些私有方法调用可能会碰到的审核不通过问题,正好上面那篇文章里提供了思路,用runtime把三个私有方法重定向到自定义的方法上(方法名只是把”_“去掉了)。

+ (BOOL) resolveInstanceMethod:(SEL) selector

{

NSString * name = NSStringFromSelector(selector);

if ( [name hasPrefix:@"_"] )

{

name = [name substringFromIndex:1];

SEL aSelector = NSSelectorFromString(name);

Method method = class_getInstanceMethod(self, aSelector);

if ( method )

{

class_addMethod(self,

selector,

method_getImplementation(method),

method_getTypeEncoding(method));

return YES;

}

}

return [super resolveInstanceMethod:selector];

}

至此大功告成,问题顺利解决。但这个问题之所以会出现的真正缘由我还是没有弄清楚,Apple在iOS7下究竟做了什么,哪位大神如果知道的话还请告知。。感激不尽 ~_~。

附上修改过后的 ASIInputStream 代码

后记

问题没解决的时候其他同事建议我更换成AFNetworking等等其他开源库,因为这些库更新快文档全,ASIHTTPRequest接口复杂、代码繁多而且如今已年久失修无人维护了。确实如此,但由于一些项目上的原因我们无法更换,况且ASIHTTRequest在效率上略好,并且拥有其他基于NSURLConnection的库不具备一些功能。在解决问题的过程中也让我对NSInputStream的工作机制有了更深的理解,可谓一石二鸟。由于鄙人能力有限,其中一些地方可能说的有错误或者有纰漏的话还请大神们指正:

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 211,265评论 6 490
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 90,078评论 2 385
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 156,852评论 0 347
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,408评论 1 283
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,445评论 5 384
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,772评论 1 290
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,921评论 3 406
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,688评论 0 266
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,130评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,467评论 2 325
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,617评论 1 340
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,276评论 4 329
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,882评论 3 312
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,740评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,967评论 1 265
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,315评论 2 360
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,486评论 2 348

推荐阅读更多精彩内容