最近,收到两家大客户反馈的bug,都是我们android版本sdk报的bug。既然大客户给我们报bug了,那必须十分重视对待。
第一个bug,SecurityException: Permission denied (missing INTERNET permission?)
初看一下以为是网络权限的问题,难道客户没有添加网络的权限,这显然不可能。
从stackoverflow上搜到这篇文章很有帮助。大致了解出现这个bug的原因。
首先,android.permission.INTERNET并不是唯一的网络相关权限,还需要声明一下android.permission.ACCESS_NETWORK_STATE这个权限。这个权限对于sdk而言,我还是很纠结地,因为很多客户不一定会使用到这个权限。如果我多添加了这个权限的判断会导致一些客户无法使用sdk。
其次,在root过的设备上,用户可以通过工具来修改在运行时授予已安装应用的权限。同时,在某些定制的rom上也可以这么做的。
再次,Android 6.0以后引入了全新的权限管理机制,称为运行时权限。它允许用户更多地控制授予的权限(也允许选择性授予),或允许一个撤销已授予的权限,无需应用程序删除。
知道这个bug产生的原因之后,那我们来解决问题吧。
解决方案:
在调用网络框架之前,把请求网络的request放入框架中http请求的队列之前,先判断一下权限是否已经添加:
<uses-permission android:name="android.permission.INTERNET" />
public void addToRequestQueue(Request req) {
if (!Util.checkPermission(MWConfiguration.getContext(), "android.permission.INTERNET")) {
return;
}
getRequestQueue().addRequest(req);
}
添加了完权限的判断之后,可以把影响降到最低。
第两个bug,java.lang.RuntimeException: Package manager has died
这个bug也很奇葩,是这段代码造成的
/**
* 获得当前应用的版本号
*
* @param context context
* @return App Version
*/
public static String getAppVersion(Context context) {
String result = "1.0";
PackageInfo info = null;
try {
info = context.getPackageManager().getPackageInfo(context.getPackageName(), 0);
} catch (NameNotFoundException ignored) {
}
if (info != null) {
result = info.versionName;
}
return result;
}
初看一下,没有任何问题。但为何会引起了异常呢?
它是一个Binder调用,造成这个的原因是因为发生了RemoteException。
如果Binder的使用超出了一个进程的限制就会抛TransactionTooLargeException这个异常。
通过查看源码发现,在一个进程中Binder的大小大约是1M。如果这时候恰巧在用getPackageManager()做其他事情,就会提示Package manager has died。
解决方案:
第一种办法,在getAppVersion()方法前加上synchronized。其他用到context.getPackageManager()的地方也尽量加上synchronized
第二种办法,第一次获取完appVersion这种数据后放在一个地方缓存起来。以后每次直接读取缓存中的数据。
总结
写sdk的上辈子都是折翼天使,希望以后再也不要写sdk了,让我先去哭一会儿去。当然这些是不可能的,写代码还是必须严谨再严谨啊。