writeInt, writeUint checks

Re2906 · Jul 31, 2023 · 448ac6f · 448ac6f
1 parent 6c1b8cb
commit 448ac6f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/dist/tonweb.js b/dist/tonweb.js
diff --git a/src/boc/BitString.js b/src/boc/BitString.js
@@ -115,6 +115,11 @@ class BitString {
      * @param bitLength  {number}  size of uint in bits
      */
     writeUint(number, bitLength) {
+        if (isNaN(number)) throw new Error('writeUint: value is NaN');
+        if (number === null) throw new Error('writeUint: value is null');
+        if (number === undefined) throw new Error('writeUint: value is undefined');
+        if (!bitLength) throw new Error('writeUint: no bitLength');
+
         number = new BN(number);
         if (
             bitLength == 0 ||
@@ -135,6 +140,11 @@ class BitString {
      * @param bitLength  {number}  size of int in bits
      */
     writeInt(number, bitLength) {
+        if (isNaN(number)) throw new Error('writeInt: value is NaN');
+        if (number === null) throw new Error('writeInt: value is null');
+        if (number === undefined) throw new Error('writeInt: value is undefined');
+        if (!bitLength) throw new Error('writeInt: no bitLength');
+
         number = new BN(number);
         if (bitLength == 1) {
             if (number == -1) {

diff --git a/src/contract/wallet/WalletContract.js b/src/contract/wallet/WalletContract.js
@@ -16,7 +16,7 @@ class WalletContract extends Contract {
 
         this.methods = {
             /**
-             * @param   params {{secretKey: Uint8Array, toAddress: Address | string, amount: BN | number, seqno: number, payload: string | Uint8Array | Cell, sendMode: number, stateInit?: Cell, expireAt?: number}}
+             * @param   params {{secretKey: Uint8Array, seqno: number, expireAt?: number, toAddress: Address | string, amount: BN, payload?: string | Uint8Array | Cell, sendMode?: number, stateInit?: Cell }}
              */
             transfer: (params) => Contract.createMethod(provider, this.createTransferMessage(params.secretKey, params.toAddress, params.amount, params.seqno, params.payload, params.sendMode, !Boolean(params.secretKey), params.stateInit, params.expireAt)),
 
@@ -221,7 +221,7 @@ class WalletContract extends Contract {
         if (messages.length < 1 || messages.length > 4) {
             throw new Error('put 1-4 messages');
         }
-        for (let msg of messages) {
+        for (const msg of messages) {
             let sendMode = msg.sendMode;
             if (sendMode === null || sendMode === undefined) {
                 sendMode = 3;