Android WebView安全方面的一些坑

公司一款app有将近两年没有更新了，虽然用户量不大，但是因为与第三方有合作，出现问题时需要进行维护；没想到最近第三方对他们所有的软件进行了网络安全扫描，这款Android app也未能幸免…

因为app是13年左右开发的，维护也只是到16、17年左右就终止了，所以，扫描出不少漏洞；因为是采用了webview+html混合开发，因此，需要解决一些webview相关的问题：

一、webview隐藏接口问题（任意命令执行漏洞）

android webview组件包含3个隐藏的系统接口：searchBoxJavaBridge_, accessibilityTraversal以及accessibility，恶意程序可以通过反射机制利用它们实现远程代码执行；该问题在Android4.4以下版本出现。 于是，在Android3.0到4.4之间的版本，我们通过移除这些隐藏接口，来解决该问题：

 // 194.4Build.VERSION.KITKAT// 113.0Build.VERSION_CODES.HONEYCOMBif(Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB && Build.VERSION.SDK_INT < 19 && webView != null) { webView.removeJavascriptInterface("searchBoxJavaBridge_");webView.removeJavascriptInterface("accessibility");webView.removeJavascriptInterface("accessibilityTraversal");} 

二、addJavascriptInterface任何命令执行漏洞

在webview中使用js与html进行交互是一个不错的方式，但是，在Android4.2(16，包含4.2)及以下版本中，如果使用addJavascriptInterface，则会存在被注入js接口的漏洞；在4.2之后，由于Google增加了@JavascriptInterface，该漏洞得以解决。

解决该问题，最彻底的方式是在4.2以下放弃使用addJavascriptInterface，采用onJsPrompt或其它方法替换。或者使用一些方案来降低该漏洞导致的风险：如使用https并进行证书校验，如果是http则进行页面完整性校验，如上面所述移除隐藏接口等。

 public boolean onJsPrompt(WebView view, String url, String message,String defaultValue, JsPromptResult result) {result.confirm(CGJSBridge.callJava(view, message));Toast.makeText(view.getContext(),"message="+message,Toast.LENGTH_LONG).show();return true;} 

三、绕过证书校验漏洞

webviewClient中有onReceivedError方法，当出现证书校验错误时，我们可以在该方法中使用handler.proceed()来忽略证书校验继续加载网页，或者使用默认的handler.cancel()来终端加载。 因为我们使用了handler.proceed()，由此产生了该“绕过证书校验漏洞”。 如果确定所有页面都能满足证书校验，则不必要使用handler.proceed()

 @SuppressLint("NewApi")@Overridepublic void onReceivedSslError(WebView view, SslErrorHandler handler, SslError error) {//handler.proceed();// 接受证书super.onReceivedSslError(view, handler, error);} 

四、allowFileAccess导致的File域同源策略绕过漏洞

如果webview.getSettings().setAllowFileAccess(boolean)设置为true，则会面临该问题；该漏洞是通过WebView对Javascript的延时执行和html文件替换产生的。 解决方案是禁止WebView页面打开本地文件，即

 webview.getSettings().setAllowFileAccess(false); 

或者更直接的禁止使用JavaScript

 webview.getSettings().setJavaScriptEnabled(false);