一个教训

这是一个关于targetSdkVersion的教训。

之前有用户反馈,在android12设备上,会报蓝牙权限适配导致的crash:

Caused by: Java.lang.SerurityException: Need android.permission.BlUETOOTH_CONNECT permission for android.content.AttributionSource@70c2370b: getName

看了下,Android 12上将原来的android.permission.BLUETOOTH权限拆分成了三个:
BLUETOOTH_SCANBLUETOOTH_ADVERTISEBLUETOOTH_CONNECT
看了下适配文档,将SDK的AndroidManifest里的蓝牙权限做了如下修改:

- <uses-permission android:name="android.permission.BLUETOOTH" />
+ <uses-permission android:name="android.permission.BLUETOOTH"
+    android:maxSdkVersion="30" />
+ <uses-permission android:name="android.permission.BLUETOOTH_CONNECT" />

然后,Java代码里做了些相关改动。正好办公室也借不到Android12的设备。目测了下代码,感觉没啥问题,就提交了。

几个星期后,有客户反馈,Android 12上蓝牙,报蓝牙权限缺失导致的crash:

java.lang.SecurityException: Need BLUETOOTH permission
at android.bluetooth.BluetoothHeadset.<init>(BluetoothHeadset.java:426)
at android.bluetooth.BluetoothAdapter.getProfileProxy(BluetoothAdapter.java:3139)

看了下源码,crash在下面代码:

// Preserve legacy compatibility where apps were depending on
// registerStateChangeCallback() performing a permissions check which
// has been relaxed in modern platform versions
if (context.getApplicationInfo().targetSdkVersion <= Build.VERSION_CODES.R
         && context.checkSelfPermission(android.Manifest.permission.BLUETOOTH)
                  != PackageManager.PERMISSION_GRANTED) {
       throw new SecurityException("Need BLUETOOTH permission");
}

也就是在Android12设备上运行的APP,如果targetSdkVersion小于31并且没有android.Manifest.permission.BLUETOOTH这个权限,运行到这里,就会抛出蓝牙权限缺失异常。
目前这个阶段,大部分客户的APP应该是没有适配Android12的,那么targetSdkVersion肯定是小于31,然后升级了之前我改动过的SDK,没有android.permission.BLUETOOTH这个权限,导致crash。

这是第一个教训,没有经过验证的代码就有可能存在bug。

然后客户又说,老的SDK打包的APK在Android 12上可以运行,并且蓝牙相关功能也正常。
当时就觉得奇怪,既然你运行正常,那第一个客户是怎么crash的?

但是由于比较匆忙,也没细想,直接在相关代码里加了这么一个判断,规避了crash。

if (Build.VERSION.SDK_INT < 31 || context.getApplicationInfo().targetSdkVersion < 31) {
      return checkPermission(context, BLUETOOTH); //对老权限校验,不通过则不往下执行。
}

于是又犯了第二个错误,没有理清bug的根源就胡乱修改。

后面越想越不对,新Feature不都应该是前向兼容的么?新的设备上运行targetSdkVersion还比较老的APP应该是没有问题才对。而且我的第二个修改会导致targetSdkVersion < 31的APP在android 12上无法运行,因为AndroidManifest里的权限是新的:

<uses-permission android:name="android.permission.BLUETOOTH" />
<uses-permission android:name="android.permission.BLUETOOTH"
    android:maxSdkVersion="30" />
<uses-permission android:name="android.permission.BLUETOOTH_CONNECT" />

我的第二个修改肯定是不对的。而且Google肯定也不会犯这么低级的错误。

那么第一个客户又是怎么crash的呢?

于是又去询问第一个用户的,问他targetSdkVersion是多少。

果然,他说是31。这下真相大白,targetSdkVersion指向了Android12,但是有没有做相应的代码适配,crash是必然。这个适配操作理应由客户端完成,而不是SDK。

于是,将SDK两次修改的代码全部回退,至此,问题告一段落。

好在我们的客户量不大, 不然上线的错误修改势必会导致大规模用户使用异常,这是灾难级别的。

以后去大厂了可不能犯这种低级错误,要理清问题根源在修改代码,牢记牢记。

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

推荐阅读更多精彩内容