diff --git a/src/index.ts b/src/index.ts
old mode 100644
new mode 100755
index 421cd71..8d9a902
--- a/src/index.ts
+++ b/src/index.ts
@@ -263,16 +263,21 @@ export default class Wallet {
    * @param icapDirect setting this to `true` will generate an address suitable for the `ICAP Direct mode`
    */
   public static generate(icapDirect: boolean = false): Wallet {
-    if (icapDirect) {
-      const max = new BN('088f924eeceeda7fe92e1f5b0fffffffffffffff', 16)
-      while (true) {
+    // Note that randomBytes(32) can not pass isValidPrivate with very low probability
+    // see more https://raw.githubusercontent.com/paulmillr/noble-secp256k1/master/test/vectors/privates.json
+    while (true) {
+      if (icapDirect) {
+        const max = new BN('088f924eeceeda7fe92e1f5b0fffffffffffffff', 16)
         const privateKey = randomBytes(32) as Buffer
         if (new BN(privateToAddress(privateKey)).lte(max)) {
+          if (!isValidPrivate(privateKey)) continue
           return new Wallet(privateKey)
         }
+      } else {
+        const privateKey = randomBytes(32) as Buffer
+        if (!isValidPrivate(privateKey)) continue
+        return new Wallet(privateKey)
       }
-    } else {
-      return new Wallet(randomBytes(32))
     }
   }
 
@@ -288,7 +293,7 @@ export default class Wallet {
       const privateKey = randomBytes(32) as Buffer
       const address = privateToAddress(privateKey)
 
-      if (pattern.test(address.toString('hex'))) {
+      if (pattern.test(address.toString('hex')) && isValidPrivate(privateKey)) {
         return new Wallet(privateKey)
       }
     }