Fixed restriction USB.getSerialNumber

CHANGELOG.md

@@ -22,6 +22,7 @@ Changelog
 **Next release**
 
 * Prevent accidental application name clicks ([issue](/../../issues/2109))
+* Fixed restriction *USB.getSerialNumber*
 
 [Open issues](https://github.com/M66B/XPrivacy/issues?state=open)
 

src/biz/bokhorst/xprivacy/XUsbDevice.java

@@ -4,7 +4,6 @@
 import java.util.List;
 
 import android.os.Binder;
-import android.util.Log;
 import biz.bokhorst.xprivacy.XHook;
 
 public class XUsbDevice extends XHook {
@@ -40,25 +39,27 @@ public static List<XHook> getInstances() {
 
 	@Override
 	protected void before(XParam param) throws Throwable {
-		if (mMethod == Methods.getDeviceId) {
+		// Do nothing
+	}
+
+	@Override
+	protected void after(XParam param) throws Throwable {
+		switch (mMethod) {
+		case getDeviceId:
 			if (param.args.length > 0 && param.args[0] instanceof String) {
 				if (isRestrictedExtra(param, (String) param.args[0]))
 					param.setResult(0);
 			} else {
 				if (isRestricted(param))
 					param.setResult(0);
 			}
+			break;
 
-		} else if (mMethod == Methods.getDeviceName) {
-			if (isRestricted(param))
+		case getDeviceName:
+		case getSerialNumber:
+			if (param.getResult() != null && isRestricted(param))
 				param.setResult(PrivacyManager.getDefacedProp(Binder.getCallingUid(), "USB"));
-
-		} else
-			Util.log(this, Log.WARN, "Unknown method=" + param.method.getName());
-	}
-
-	@Override
-	protected void after(XParam param) throws Throwable {
-		// Do nothing
+			break;
+		}
 	}
 }