sandeel / Bushido-Blocks

GNU General Public License v3.0
56 stars 20 forks source link

API Compatibility Issues #2

Open lilicoding opened 7 years ago

lilicoding commented 7 years ago

Dear developers,

We have recently developed a state-of-the-art static analysis tool for uncovering API compatibility issues in Android apps. Applying this tool to open source apps on F-droid, we have exposed a few instances of compatibility issues and submitting them to development teams for a fix.

For your app, we have found that this project has accessed the following APIs which are available only on an API level higher than the declared minSdkVersion and which are accessed without proper protection. In other words, if those APIs get called at runtime, it will trigger a NoSuchMethodError and thus result in a crash of the running application.

<android.service.wallpaper.WallpaperService.Engine: void onDestroy()>:[7,25] <android.service.wallpaper.WallpaperService.Engine: android.view.SurfaceHolder getSurfaceHolder()>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onSurfaceDestroyed(android.view.SurfaceHolder)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void setTouchEventsEnabled(boolean)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onVisibilityChanged(boolean)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: boolean isPreview()>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onSurfaceChanged(android.view.SurfaceHolder,int,int,int)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onOffsetsChanged(float,float,float,float,int,int)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: android.os.Bundle onCommand(java.lang.String,int,int,int,android.os.Bundle,boolean)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onCreate(android.view.SurfaceHolder)>:[7,25] <android.service.wallpaper.WallpaperService.Engine: void onSurfaceCreated(android.view.SurfaceHolder)>:[7,25]

Note that, because of the nature of the static analysis, we cannot confirm whether the flagged APIs would actually be called at runtime (e.g., unreachable code). However, we still believe that those APIs, which may cause compatibility issues, should not be accessed or at least be accessed with proper protections.

In addition to the aforementioned APIs (i.e., backward-compatibility), which could cause app crashes if accessed, we have also identified that this project has also accessed some APIs that have been removed from the latest public SDK, making the app possibly suffer from forward-compatibility issues.

<java.lang.reflect.Array: java.lang.Object newInstance(java.lang.Class,int[])>:[1,8]

We would be very much appreciated if you can acknowledge to us that those reported APIs are indeed problematic for the project’s long-term stability. please let us know if you need any more information relating to this issue report.