NSInvocation setReturnValue 为NSObject时Bad Access问题

示例代码可参照:hotacool/aspect 的AspectsDemoTests

问题概述

需求:ARC模式下,使用aspect,hook一个返回数组的实例方法,像原返回数组中插入一条数据。如下代码,需要getArray返回@[@"222", @"111"]

// test class:
@implementation TestClass
- (NSArray*)getArray {
    return @[@"111"];
}
@end
// aspect hook
id aspect = [TestClass aspect_hookSelector:@selector(getArray)
                                   withOptions:AspectPositionInstead
                                    usingBlock:^(id<AspectInfo> info){
                                        NSArray *originRetValue;
                                        [[info originalInvocation] invoke];
                                        [[info originalInvocation] getReturnValue:&originRetValue];
                                        NSMutableArray *retValue = [NSMutableArray arrayWithArray:originRetValue];
                                        [retValue insertObject:@"222" atIndex:0];
                                        [[info originalInvocation] setReturnValue:&retValue];
                                    }
                                         error:nil];

问题: 上面aspect hook方式肯定是错误的,会crash报错EXC_BAD_ACCESS。

原因分析

  1. NSInvocation setReturnValue 是指针操作
    首先我们知道,NSInvocation的setReturnValue是指针操作:
- (void)setReturnValue:(void *)retLoc;
- (void)setArgument:(void *)argumentLocation atIndex:(NSInteger)idx;

对于基础数据类型的变量,我们可以直接&var取指针地址:

id aspect = [testClass aspect_hookSelector:@selector(callReturnsDouble) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info){
        double toReturn = 3.5;
        void *ptr = &toReturn;
        [info.originalInvocation setReturnValue:ptr];
    }error:NULL];

由于基础数据类型的内存管理上的特殊性,不用担心被提前释放,造成野指针的情况。同理,NSString虽然继承于NSObject,但由于编译器的内部优化,使用语法糖形式(@"xxx")而不通过alloc方式创建时,并没有分配内存到堆区,和基础类型一样可以通过如下方式操作:

// testInsteadHook: test succeed
id aspect = [testObject aspect_hookSelector:@selector(name) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info) {
        NSString *customText = @"Custom Text";
        [[info originalInvocation] setReturnValue:&customText];
    } error:NULL];
// alloc 方式会crash
id aspect = [testObject aspect_hookSelector:@selector(name) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info) {
        NSString *customText = [[NSString alloc] initWithUTF8String:"(nonnull const char *)"];
        [[info originalInvocation] setReturnValue:&customText];
    } error:NULL];
  1. ARC模式下,对象创建后,出了autorelease pool会被自动释放
    回到我们的问题,ARC模式下会为block范围创建autorelease pool,当在aspect的block中创建新的数组对象,会将对象自动retain到堆区,当block结束,对象自动释放,当invocation内部再调用到这个对象时,会出现EXC_BAD_ACCESS crash。
    作为验证,我们可以在外部保存对象的内存不释放,不会导致crash:
NSString *__block customText;//内存不释放,不会crash
    id aspect = [testObject aspect_hookSelector:@selector(name) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info) {
        customText = [[NSString alloc] initWithUTF8String:"(nonnull const char *)"];
        [[info originalInvocation] setReturnValue:&customText];
    } error:NULL];

    NSLog(@"%@", testObject.name);

解决方案

保持对象的内存不释放,当然不是个好方案。我们需要使新增的对象跟origin method内部的对象一样的内存生命周期。这里分两种不同需求情况:

  1. 不需要更改或者新增返回对象的内存地址
    如果不需要改变origin method内部的返回对象内存地址,即对象内存指针,我们可以直接获取origin return value的指针并返回,这里我们使用TestClass做例子而不能用NSString:
// crash with EXC_BAD_ACCESS
TestClass *testObject = [TestClass new];
    id aspect = [TestClass aspect_hookSelector:@selector(getArray) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info) {
        NSArray *originRetValue;
        [[info originalInvocation] invoke];
        [[info originalInvocation] getReturnValue:&originRetValue];
        [[info originalInvocation] setReturnValue:&originRetValue];
    } error:NULL];
    NSLog(@"%@", [testObject getArray]);

直接使用NSArray *originRetValue的指针肯定是crash的,因为它在block内部被retain,出了block被释放,会出现野指针。为了不提前释放,我们可以使用__unsafe_unretained来表明该变量不会retain对象:

// test succeed
TestClass *testObject = [TestClass new];
    id aspect = [TestClass aspect_hookSelector:@selector(getArray) withOptions:AspectPositionInstead usingBlock:^(id<AspectInfo> info) {
        __unsafe_unretained NSArray * originRetValue;
        [[info originalInvocation] invoke];
        [[info originalInvocation] getReturnValue:&originRetValue];
        [[info originalInvocation] setReturnValue:&originRetValue];
    } error:NULL];
    NSLog(@"%@", [testObject getArray]);

这里要理解__unsafe_unretained修饰词,__unsafe_unretained 跟主要跟 C 代码交互,跟weak功能相似,不会retain对象,但是它没有weak自动设置nil的功能。所以你必须准确知道修饰对象的内存生命周期,确保不会因为提前释放,而导致调用野指针。
在我们这里,它表示originRetValue不会retain origin method内部返回值,返回值还是它原来的内存管理,所以出了block,仍然可以完成invocation的使用。

  1. 需要改变返回对象的内存地址
    回到本例,我们显然需要改变内存地址,NSArray不可变,无法满足修改的要求。这里笔者没有在网上找到合理的的解决方案,目前的方案是,直接绑定新增对象到origin return value,与原返回值同生命周期:
- (void)testObjectInsteadHook {
    TestClass *testObject = [TestClass new];
    XCTAssertEqualObjects([testObject getArray], @[@"111"], @"Must match");
    id aspect = [TestClass aspect_hookSelector:@selector(getArray)
                                   withOptions:AspectPositionInstead
                                    usingBlock:^(id<AspectInfo> info){
                                        __unsafe_unretained NSArray *originRetValue;
                                        [[info originalInvocation] invoke];
                                        [[info originalInvocation] getReturnValue:&originRetValue];
                                        NSMutableArray *retValue = [NSMutableArray arrayWithArray:originRetValue];
                                        __block BOOL isContain = NO;
                                        [retValue enumerateObjectsUsingBlock:^(NSString * _Nonnull obj, NSUInteger idx, BOOL * _Nonnull stop) {
                                            if ([obj isEqualToString:@"222"]) {
                                                isContain = YES;
                                                *stop = YES;
                                            }
                                        }];
                                        if (!isContain) {
                                            [retValue insertObject:@"222" atIndex:0];
                                        }
                                        objc_setAssociatedObject(originRetValue, "kRetValueKey", retValue, OBJC_ASSOCIATION_RETAIN_NONATOMIC);//与originRetValue同生命周期
                                        [[info originalInvocation] setReturnValue:&retValue];
                                    }
                                         error:nil];
    XCTAssertEqualObjects([testObject getArray], (@[@"222", @"111"]), @"Must match");
    XCTAssertTrue([aspect remove], @"Must return YES");
    XCTAssertEqualObjects([testObject getArray], @[@"111"], @"Must match");
    XCTAssertFalse([aspect remove], @"Must return NO");
}

这样来保持对象的释放与原返回对象同步,不会提前释放,同时objc_setAssociatedObject的对象会在target释放时自动释放,避免leak。

参考:
stackoverflow
__unsafe_unretained

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

推荐阅读更多精彩内容